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

[Nicolas George]

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.

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200628/3f59798d/attachment.sig>