Cool. And we contact you when samba-PIDL no longer can generate compileable wireshark dissectors?
On Thu, Oct 3, 2013 at 7:13 PM, Andrew Bartlett <abart...@samba.org> wrote: > On Tue, 2013-10-01 at 10:55 -0400, mman...@netscape.net wrote: >> The check_col function in the Wireshark source has been deprecated for >> awhile, but never officially removed. I've been modifying the source >> (removing calls) so that it can be officially removed. The last big hurdle >> was the DCE/RPC generated dissector files. I've had a difficult time trying >> to regenerate the dissectors with the necessary modified source, so I just >> manually updated dissectors to mimic what the source should generate. >> Attached is a patch that contains the modifications to "PIDL source" >> that generates the dissector files (and not the dissector files >> themselves). I couldn't tell from the Wireshark wiki [1] if this was >> maintained by samba or Wireshark, so now both will now have a copy, >> with the source already committed to Wireshark repository [3]. >> >> Patch is also available in Wireshark bugzilla [3]. >> >> Michael Mann >> >> [1] http://wiki.wireshark.org/Pidl >> [2] http://anonsvn.wireshark.org/viewvc?view=revision&revision=52313 >> [3] https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8804 > > PIDL is maintained by the Samba team, the canonical repository is in our > master branch of git://git.samba.org/samba.git > > From the Samba perspective: > > I would warn that patching the generated code only passes the pain of > fixing this properly down to the next person who needs to work here. > > As improvements are made to the IDL files or the parser, the generated > files need to be re-generated, and then someone has the un-enviable > choice of throwing away manual changes, or reverse engineering these > back into the compiler. > > Even if you think you have modified them correctly to match your PIDL > changes, please persevere to get the files re-generated using PIDL. > Only then will we really know the generator generates the code > correctly, and only then will the solution be long-term maintainable. > > Thanks, > > Andrew Bartlett > > -- > Andrew Bartlett > http://samba.org/~abartlet/ > Authentication Developer, Samba Team http://samba.org > Samba Developer, Catalyst IT http://catalyst.net.nz > > ___________________________________________________________________________ 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