[vlc-devel] [PATCH 2/5] Qt: PrefsItemData search

Yuval Tze yuvaltze at gmail.com
Wed Jul 27 20:25:23 CEST 2011


I meant help :)
head could be name or long name, I should have mentioned that most of the
weird code here was taken from the AdvPrefsPanel constructor, since it goes
over the module the same way but without actually creating the controls..

On Wed, Jul 27, 2011 at 3:25 AM, Jean-Baptiste Kempf <jb at videolan.org>wrote:

> On Wed, Jul 27, 2011 at 12:17:34AM +0300, Yuval Tze wrote :
> > add contains function to search text in item's name, help and in the
> config items' text
>
> > +/* go over the module config items and search text in psz_text
> > + * also search the module name and head */
>
> head? You mean longname?shortname?desc?
>
> > +            if(  p_item->i_type == CONFIG_SUBCATEGORY &&
> > +                            ( ( this->i_type == TYPE_SUBCATEGORY &&
> > +                              p_item->value.i == this->i_object_id ) ||
> > +                            ( this->i_type == TYPE_CATSUBCAT &&
> > +                              p_item->value.i == this->i_subcat_id ) ) )
>
> Weird parenthesis placement or indentation.
>
> Else, ok.
>
> Best Regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110727/5767d4a9/attachment.html>


More information about the vlc-devel mailing list