Wireshark-dev: Re: [Wireshark-dev] MPEG2-TS, DVB-SI, and DVB-GSE Dissectors

From: Jaap Keuter <jaap.keuter@xxxxxxxxx>
Date: Sat, 18 Mar 2017 23:14:20 +0100
On 18-03-17 21:11, Alexander Adolf wrote:
> Hello Martin,
> 
> On 2017-03-17, at 16:22 , Martin Kaiser <lists@xxxxxxxxx> wrote:
> 
>> [...]
>> good to see you again. We worked together in the DVC-CI+ group back in
>> 2014.
> 
> Good seeing you again! I believe to remember it was the meeting in Munich?
> 
>> I hope that I told you about Wireshark's CI+ dissector then ;-)
> 
> Yu did indeed. ;)
> 
>> [...]
>> There's no such thing as a "high-level introduction to the DVB-related
>> functionality" document.
> 
> Which I hadn't expected anyway.
> 
>> There is, however, doc/README.dissector, which is worth a read.
> 
> I'm studying this at the moment. But there's always the moment when you're convinced you had followed all instructions meticulously, but it still won't work. That's when a little hint from a more experienced developer comes handy.
> 
>>> (1) A little bit of streamlining of how DVB PSI/SI is presented when
>>> analysing MPEG2-TS. I could e.g. imagine it to help readability if the
>>> PID were shown in the source and/or destination address field. Also,
>>> when interpreting DVB PSI/SI tables, some interpretations of
>>> identifiers doesn't seem to be quite accurate. 
>>
>> That sounds like a good starting point: Find a field that we don't
>> interpret correctly. Try to find out where it's defined and modify it.
>> Try to come up with a small and simple modification for the first patch.
>> If you get stuck along the way, let me know which field you're looking
>> at and prepare a ts file that you can share with us.
>> [...]
> 
> Sounds like a plan. My first patch should be... posted here?
> 
> 

Post here? No, submit to Gerrit: https://code.wireshark.org/review
Why? because: https://wiki.wireshark.org/Development/Workflow
With more detail here: https://wiki.wireshark.org/CreatingPatches and here:
https://wiki.wireshark.org/Development/SubmittingPatches

Regards,
Jaap