Re: [FFmpeg-devel] [PATCH 0/1] fftools/ffprobe: dump contents of the AV_FRAME_DATA_SEI_UNREGISTERED

From: Stefano Sabatini <stefasab@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Petr Matousek <petr.matousek@comprimato.com>
Subject: Re: [FFmpeg-devel] [PATCH 0/1] fftools/ffprobe: dump contents of the AV_FRAME_DATA_SEI_UNREGISTERED
Date: Wed, 3 Jan 2024 16:44:08 +0100
Message-ID: <ZZWAyGGmid4ZEf+0@mariano> (raw)
In-Reply-To: <CAK+ULv7BjrZw9+ZgepByahdaSk0uNg53T5BZt8P+2A-Cyvpo9A@mail.gmail.com>

On date Tuesday 2024-01-02 22:08:54 +0000, Kieran Kunhya wrote:
[...]
> The OP wants to test an implementation they have and they should do it
> using the API, not using ffprobe to test their third party implementation
> of something.
> 

> > "Parsed by FFmpeg" sounds like you are proposing to extend the FFmpeg
> > API to parse some more information (about the SEI subtype?) and make
> > it available through some more specific SEI side data?
> >
> 
> As the OP says, this SEI data is from "MISB ST 0604" which is something
> that could be implemented in FFmpeg and parsed properly like all the other
> side data instead of just writing random binary data to the terminal.

Fine, so you are proposing to extend the parser to put this
information in the side data. This makes sense to me and sounds a
better approach than the original proposal.

@Petr can you update the patch to extend the parser to expose this
information? Feel free to ask in case something is not clear (I'm not
familiar myself with that part of the code, but I can try to help).
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".