Hi again,
no problem. I have already started to move a good amount of the
literals into the h-file and I will try to move all I can into
"value_string" arrays.
But this won't be possible for everything, as MIDI itself has
some "specialities" already within the coding and RTP-MIDI takes
this even one step further...
So I won't get around the use of a couple of "add_text()" calls.
Apart from that, I noticed quite a few other plugins to use the
"add_text()" pretty regularly - so what's the reason not to use it?
Tobias
> -----Ursprüngliche Nachricht-----
> Von: wireshark-dev-bounces@xxxxxxxxxxxxx
> [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag von
> Jaap Keuter
> Gesendet: Sonntag, 22. Oktober 2006 12:35
> An: Developer support list for Wireshark
> Betreff: Re: [Wireshark-dev] Wireshark 0.99.4pre1 is now
> available [heur]
>
> Hi,
>
> Sorry dude, since it's not yet picked up and we want no 'big'
> changes now I vote against it. The reason why it was not
> picked up? I don't know, but I would have to rework the code
> considerable before committing it, as I would want to get rid
> of the 10 level deep conditionals, the use of literals iso
> constants and the use of add_text(). But that's just my preference.
>
> Thanx,
> Jaap
>
> On Sat, 21 Oct 2006, Tobias Erichsen wrote:
>
> > Hi Gerald,
> >
> > how about including my RTP-MIDI dissector plugin in this
> new version?
> >
> > Best regards,
> > Tobias
> >
> > > -----Ursprüngliche Nachricht-----
> > > Von: wireshark-dev-bounces@xxxxxxxxxxxxx
> > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag
> von Gerald
> > > Combs
> > > Gesendet: Samstag, 21. Oktober 2006 18:55
> > > An: Wireshark announcements; Developer support list for
> Wireshark;
> > > Community support list for Wireshark
> > > Betreff: [Wireshark-dev] Wireshark 0.99.4pre1 is now available
> > > [heur]
> > >
> > > Wireshark 0.99.4pre1 is now available for testing.
> Source code and
> > > a Windows installer can be downloaded immediately from
> > >
> > > http://www.wireshark.org/download/prerelease/wireshark-0.99.4p
> > > re1.tar.gz
> > > http://www.wireshark.org/download/prerelease/wireshark-setup-0
> > > .99.4pre1.exe
> > >
>
> _______________________________________________
> Wireshark-dev mailing list
> Wireshark-dev@xxxxxxxxxxxxx
> http://www.wireshark.org/mailman/listinfo/wireshark-dev
>