[FFmpeg-devel] [PATCH v2 1/2] libavcodec/pgxdec: Add PGX decoder

[Gautam Ramakrishnan]

On Sun, Jun 28, 2020 at 4:21 PM Nicolas George <george at nsup.org> wrote:
>
> Gautam Ramakrishnan (12020-06-28):
> > I shall change this. I do not understand how the check can be
> > removed. It is compulsory that a PGX file must have these bytes
>
> In case it was not clear, I think Carl Eugen was suggesting you include
> the string directly in the comparison.
>
This was clear.
> But you do not need that test, because you do not need the file to be a
> 100% valid PGX file. It is not the task of a decoder to enforce
> conformance: its task is to decode.
>
This is what I do not understand. Suppose the file does not start
with the bytes "PG ML ", how is that to be handled?




-- 
-------------
Gautam |