The problem is that all "external" expert APIs end up calling static functions within expert.c and I really don't want to have that much copy/paste for something that will hopefully be removed shortly. Since the word is now out through the -commit and -dev mailing lists, people should consider themselves warned. The convert_expert_add_info_format.pl script really helped me get the whole conversion done much faster than I anticipated, so people can certainly email me (or -dev) with questions/issues they have with its use.
-----Original Message----- From: Evan Huus <eapa...@gmail.com> To: wireshark-dev <wireshark-dev@wireshark.org> Sent: Sun, Sep 8, 2013 3:49 pm Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 51844: /trunk/epan/ /trunk/epan/: expert.c expert.h proto.c /trunk/epan/wslua/: init_wslua.c wslua_tree.c Excellent :) Suggest simply move the prototype for _internal into the two C files that use it, so it's not even available for anyone who shouldn't have it? Kind of yucky though, it's true... On 2013-09-08, at 7:32 PM, mm...@wireshark.org wrote: > http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=51844 > > User: mmann > Date: 2013/09/08 12:32 PM > > Log: > Couldn't quite figure out how to make the expert items in proto.c and the LUA API filterable (suggestions welcome!), but I wanted to start forcing dissectors to use the filterable expert API (nothing like compile errors to force change!). I created expert_add_info_format_internal (original expert_add_info_format) and used it as a placeholder for proto.c/LUA API until a solution is created. The "new" expert_add_info_format will be the "old" expert_add_info_format_text, and I'll slowly do the search/replace. > > For those with dissectors outside the source tree, please see tools/convert_expert_add_info_format.pl for help with the conversion. Please do not use expert_add_info_format_internal, as it's support time will be very short lived. > > Directory: /trunk/epan/ > Changes Path Action > +2 -2 expert.c Modified > +7 -4 expert.h Modified > +2 -2 proto.c Modified > > Directory: /trunk/epan/wslua/ > Changes Path Action > +2 -2 init_wslua.c Modified > +1 -1 wslua_tree.c Modified > > ___________________________________________________________________________ > Sent via: Wireshark-commits mailing list <wireshark-comm...@wireshark.org> > Archives: http://www.wireshark.org/lists/wireshark-commits > Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits > mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe