[FFmpeg-devel] [PATCH] make av_base64_encode return something useful

Luca Abeni lucabe72
Thu Jun 3 22:26:56 CEST 2010


On 03/06/10 22:11, Howard Chu wrote:
> Currently av_base64_encode() just returns the same pointer that was
> passed in for the output buffer. This is kind of a waste since the
> caller obviously already knows that value. Most callers of
> av_base64_encode() pretty much ignore its return value, or just check
> for NULL.
>
> This patch changes it to return the pointer to the end of the result,
> instead of the head of the result. This allows callers to e.g. obtain
> the length of the result without a wasted call to strlen(), or concat
> new data to the string without having to traverse again from the beginning.

The sdp.c part looks ok (but I think it should go in a separate patch / 
commit)


			Thanks,
				Luca



More information about the ffmpeg-devel mailing list