On Wed, Jun 24, 2020 at 23:57:10 +0530, gautamr...@gmail.com wrote:
> +    const char *header_start = "PG ML ";

What about LM? Or do little-endian samples not exist?

> +    if (bytestream2_peek_byte(&s->g) == ' ')
> +        bytestream2_skip(&s->g, 1);

I understand the separators can also be tabs, but that might also not
be realistic. (I'm looking at the reference parser.) Same in
pgx_get_number().

> +static inline int write_frame_8(AVPacket *avpkt, AVFrame *frame, PGXContext 
> * const s)

This is a bit similar to write_frame_16(). If a write_frame_16le() also
needs to be added, the three could perhaps be formed with macros.

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

Reply via email to