[mmann78@xxxxxxxxxxxx]

No explicit plans, it was just a copy/paste/merge of hf_ and preference functionality.  While I think the GUI could take advantage of it, I was torn as to which GUI (GTK or qt) any new functionality would be developed.  Since I don't know either that well, I remain concentrated on making all expert info filterable, so that its not confusing if new functionality comes along that only works on filterable expert info.  For example, if the expert dialog could be enhanced to take advantage of filters (and possibly not have to run redissection), I wouldn't want the "yet to be filtered" expert info to not be populated in the dialog.

 

 

-----Original Message-----
From: Evan Huus <eapache@xxxxxxxxx>
To: Wireshark Developer List <wireshark-dev@xxxxxxxxxxxxx>
Sent: Sat, Jul 6, 2013 8:28 am
Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 49559: /trunk/epan/ /trunk/epan/: epan.c expert.c expert.h packet.c proto.c

Are there plans to use the expert_modules tree added by this commit? I
was looking at switching it to wmem but realized that it's never
actually used for lookups...

Evan

On Fri, May 24, 2013 at 1:59 PM,  <mmann@xxxxxxxxxxxxx> wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=49559
>
> User: mmann
> Date: 2013/05/24 10:59 AM
>
> Log:
>  Add expert info configuration framework.  Bug 2412 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2412).
>
>  Expert info "fields" can now be registered/addressed by name.  Right now, the 
basic framework allows expert info fields to become "display filters".  However 
more could be done, like user preferences overriding default severity level, 
speeding up expert info dialog load time by not needing to redissect a file, 
etc.
>
>  Long term goal is to have all expert_info filterable and have the 
functionality of expert_add_info_format() include the "registered index".  
expert_add_info_format_text() is the workaround until all current calls to 
expert_add_info_format() have been updated with either expert_add_info() or 
expert_add_info_format_text().  Then the remaining expert_add_info_format_text() 
will be renamed to expert_add_info_format().
>
> Directory: /trunk/epan/
>   Changes    Path          Action
>   +3 -1      epan.c        Modified
>   +194 -7    expert.c      Modified
>   +77 -0     expert.h      Modified
>   +2 -2      packet.c      Modified
>   +1 -1      proto.c       Modified
>
> ___________________________________________________________________________
> Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-commits
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
>              mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe