[FFmpeg-devel] [PATCH] libavfromat/flvenc.c: Preserve the 'amf0' data stream written by Adobe Flash Media Servers while doing a data stream copy
Benedict Endemann
bendemann at make.tv
Fri Mar 8 10:02:30 CET 2013
Am 07.03.2013 19:47, schrieb Carl Eugen Hoyos:
> Benedict Endemann <bendemann <at> make.tv> writes:
>
>> - avio_w8(pb, AMF_DATA_TYPE_STRING);
>> - put_amf_string(pb, "onTextData");
>> - avio_w8(pb, AMF_DATA_TYPE_MIXEDARRAY);
>> + if (enc->codec_id == AV_CODEC_ID_TEXT) {
>> + avio_w8(pb, AMF_DATA_TYPE_STRING);
>> + put_amf_string(pb, "onTextData");
>> + avio_w8(pb, AMF_DATA_TYPE_MIXEDARRAY);
>> + avio_wb32(pb, 2);
> ...
>
> Please do not mix functional with cosmetic changes,
> ie don't re-indent so many lines but leave them
> untouched. That makes reviewing easier.
> You may send a second patch to do the indentation
> change.
>
> Thank you, Carl Eugen
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Dear Carl,
sorry it's the first patch I commited ever. Also my explanations of the
patch and what it's good for, weren't deliverd as I wanted them to.
Do you want me to send in a new "clean" patch?
Best regards, Benedict Endemann
More information about the ffmpeg-devel
mailing list