Wireshark-dev: Re: [Wireshark-dev] RTP-MIDI
From: "Tobias Erichsen" <erichsen@xxxxxxxxxxxxx>
Date: Sun, 15 Oct 2006 16:45:05 +0200
Hi again, I have changed the RTP-MIDI dissector to use the "value_string" type for retrieving textual information for controller-type and MIDI-manufacturer-ids... If there is still anything to change, let me know... Best regards, Tobias > -----Ursprüngliche Nachricht----- > Von: wireshark-dev-bounces@xxxxxxxxxxxxx > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag von > Jaap Keuter > Gesendet: Dienstag, 10. Oktober 2006 18:50 > An: Developer support list for Wireshark > Betreff: Re: [Wireshark-dev] RTP-MIDI [heur] > > Hi, > > And on top of that, could the hardcoded values be replace by > meaningfull symbols. Anders' remark may also remove the need > for all those rather atypical > text-string-printf-handling-add-type constructs. > > Thanx, > Jaap > > On Tue, 10 Oct 2006, Anders Broman (AL/EAB) wrote: > > > Hi, > > Couldn't the get_short_manu/get_long_manu constructs be > replaced by A > > const value_string static const value_string foo_vals[] = { > > { 1, "Sequential Circuits" }, > > : > > { 0, NULL } > > }; > > > > and a > > proto_tree_add_item()or roto_tree_add_uint() And a hf var Of > > { &hf_foo, > > { "Manufacturer", "rtpmidi.Manufacturer", > > FT_UINT32, BASE_DEC, VALS(foo_vals), 0, > > "Manufacturer", HFILL }}, > > > > BR > > Anders > > > > -----Original Message----- > > From: wireshark-dev-bounces@xxxxxxxxxxxxx > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Tobias > > Erichsen > > Sent: den 10 oktober 2006 10:36 > > To: Developer support list for Wireshark > > Subject: Re: [Wireshark-dev] RTP-MIDI > > > > Hi again, > > > > I have done the changes I described earlier. Attached in > the archive you find the newest source-code version of the > RTP-MIDI dissector- plugin... > > > > I would appreciate it much, if someone could check whether > this compiles & works on Linux as well... > > > > Best regards, > > Tobias > > > > > -----Ursprüngliche Nachricht----- > > > Von: wireshark-dev-bounces@xxxxxxxxxxxxx > > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag > von Anders > > > Broman > > > Gesendet: Dienstag, 10. Oktober 2006 07:16 > > > An: Developer support list for Wireshark > > > Betreff: Re: [Wireshark-dev] RTP-MIDI > > > > > > Hi, > > > At a quick glance you need to change the C++ style > comments // To /* > > > */ as all compilers do not like them. > > > Best regards > > > Anders > > > > > > -----Ursprungligt meddelande----- > > > FrÃ¥n: wireshark-dev-bounces@xxxxxxxxxxxxx > > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] För Tobias Erichsen > > > Skickat: den 8 oktober 2006 19:55 > > > Till: Developer support list for Wireshark > > > Ã"mne: Re: [Wireshark-dev] RTP-MIDI > > > > > > Well - here we go... > > > > > > Attached you will find two files: > > > > > > rtpmidi.zip -> the source of the rtp-midi plugin capture.zip > > > -> an archive which contains two captures of rtp-midi-data > > > > > > The specification that is the basis for this dissector is: > > > > http://www.cs.berkeley.edu/~lazzaro/sa/pubs/txt/current-rtp-midi.txt > > > > > > The code compiles under Windows (cygwin & vs 6), I have not tried > > > compiling it with linux, so anyone who wants to give it a try, > > > please do so. > > > > > > Additionally, I guess that the things I have done in the > dissector > > > could probably optimized. I have just used calls that I got a > > > glimpse of within some of the other dissectors (rtp & > rtp-events, rudp). > > > So comments on how to improve the code are highly welcome ;-) > > > > > > Best regards, > > > Tobias > > > > > > > > > > -----Ursprüngliche Nachricht----- > > > > Von: wireshark-dev-bounces@xxxxxxxxxxxxx > > > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag > von Tobias > > > > Erichsen > > > > Gesendet: Freitag, 6. Oktober 2006 07:48 > > > > An: Developer support list for Wireshark > > > > Betreff: Re: [Wireshark-dev] RTP-MIDI [heur] > > > > > > > > I will do some final code-cleaning over the next couple > of days, > > > > do some more testing and will post the code here some > time during > > > > the weekend... > > > > > > > > Tobias > > > > > > > > > > > > > -----Ursprüngliche Nachricht----- > > > > > Von: wireshark-dev-bounces@xxxxxxxxxxxxx > > > > > [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] Im Auftrag von > > > > Erwin Rol > > > > > Gesendet: Donnerstag, 5. Oktober 2006 23:21 > > > > > An: Developer support list for Wireshark > > > > > Betreff: Re: [Wireshark-dev] RTP-MIDI > > > > > > > > > > On Thu, 2006-10-05 at 23:15 +0200, Tobias Erichsen wrote: > > > > > > The problem is that RTP-MIDI does some "tweaking" to the > > > > MIDI-data. > > > > > > For example: System-Exclusive-data can be segmented > > > into multiple > > > > > > RTP-frames by ending a segment with a Status-Byte that > > > > > would normally not be there. > > > > > > Another thing is the fact that Realtime-Status-Bytes (just > > > > > one octet > > > > > > long) can be at any position in a "normal" > midi-stream, but in > > > > > > RTP-MIDI they always "stand alone"... > > > > > > > > > > But they still share a lot of "code", like all the command > > > > > descriptions etc. Maybe the RTP-MIDI dissector can just be > > > > extended > > > > > to also do normal MIDI (one MIDI message per packet kind). > > > > > > > > > > But anyway i hope your dissector is added soon, good work :-) > > > > > > > > > > - Erwin > > > > > > > > > > > > > > > _______________________________________________ > > > > > Wireshark-dev mailing list > > > > > Wireshark-dev@xxxxxxxxxxxxx > > > > > http://www.wireshark.org/mailman/listinfo/wireshark-dev > > > > > > > > > _______________________________________________ > > > > Wireshark-dev mailing list > > > > Wireshark-dev@xxxxxxxxxxxxx > > > > http://www.wireshark.org/mailman/listinfo/wireshark-dev > > > > > > > > > > > > > _______________________________________________ > > > Wireshark-dev mailing list > > > Wireshark-dev@xxxxxxxxxxxxx > > > http://www.wireshark.org/mailman/listinfo/wireshark-dev > > > > > _______________________________________________ > > Wireshark-dev mailing list > > Wireshark-dev@xxxxxxxxxxxxx > > http://www.wireshark.org/mailman/listinfo/wireshark-dev > > > > > > _______________________________________________ > Wireshark-dev mailing list > Wireshark-dev@xxxxxxxxxxxxx > http://www.wireshark.org/mailman/listinfo/wireshark-dev >
Attachment:
rtpmidi.zip
Description: rtpmidi.zip
- Prev by Date: [Wireshark-dev] usb dissecotr: register_dissector question
- Next by Date: [Wireshark-dev] Unaccounted bytes in USB header
- Previous by thread: Re: [Wireshark-dev] RTP-MIDI
- Next by thread: [Wireshark-dev] Your message to Wireshark-users awaits moderator approval
- Index(es):