Re: MPEG2-TS, DVB-SI, and DVB-GSE Dissectors

[Jaap Keuter <jaap.keuter () xs4all nl>]

On 18-03-17 21:11, Alexander Adolf wrote:
> Hello Martin,
>
> On 2017-03-17, at 16:22 , Martin Kaiser <lists () kaiser cx> 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

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe