[FFmpeg-devel] [PATCH 1/5] bprint: add av_bprint_init_for_buffer().

Nicolas George nicolas.george at normalesup.org
Tue Jun 5 13:51:19 CEST 2012

L'octidi 18 prairial, an CCXX, Stefano Sabatini a écrit :
> I find the "for a pre-existing buffer" expression confusing, indeed I
> couldn't tell what it means without peeking at the code.
> Maybe more clear:
> using a pre-existing buffer
> Also tell that "size" specifies the passed buffer size in bytes.

Changed the doxy into:

 * Init a print buffer using a pre-existing buffer.
 * The buffer will not be reallocated.
 * @param buf     buffer structure to init
 * @param buffer  byte buffer to use for the string data
 * @param size    size of buffer

> Same for name, maybe for_buffer -> use_buffer, but I'm not strong on
> this.

I find for_buffer more intuitive wrt my vision of how it works.


  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/20120605/97108588/attachment.asc>

More information about the ffmpeg-devel mailing list