[FFmpeg-devel] [PATCH 2/2] lavf: Add WebM DASH Manifest Muxer

Vignesh Venkatasubramanian vigneshv at google.com
Tue Jul 1 01:06:03 CEST 2014


On Mon, Jun 30, 2014 at 4:00 PM, Clément Bœsch <u at pkh.me> wrote:
> On Mon, Jun 30, 2014 at 03:44:37PM -0700, Vignesh Venkatasubramanian wrote:
> [...]
>> +static void write_string(AVFormatContext *s, const char *str, int should_free)
>> +{
>> +    avio_write(s->pb, str, strlen(str));
>> +    if (should_free)
>> +        av_free((char*)str);
>> +}
>
> Why not use avio_printf() instead of some weird jungling with
> av_asprintf() and should_free?
>

no reason. just didn't know about avio_printf. will change.

> [...]
>
> --
> Clément B.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>



-- 
Vignesh


More information about the ffmpeg-devel mailing list