[FFmpeg-devel] [PATCH 1/8] lavf: add internal demuxer helpers for subtitles.

Nicolas George nicolas.george at normalesup.org
Wed Jun 20 21:39:50 CEST 2012


Le tridi 3 messidor, an CCXX, Wolfram Gloger a écrit :
> I too would prefer the variant that is _not_ necessarily NUL-terminated
> (as for AVPacket.data).
> It seems just more natural with general ffmpeg practice.
> 'len' is the correct name then.

I agree. But there would be no harm to ensure that there is always an extra
NUL after the end of the payload, not counted in the len/size field. This
has no advert effect with binary data (except a negligible cost in memory
and time) and allows to use string-related functions in the buffer if
necessary.

> It occurs to me that your proposal is like a lightweight
> AVCodecParserContext.

I have trouble seeing the link.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120620/c19d46c5/attachment.asc>


More information about the ffmpeg-devel mailing list