[FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

Mats Peterson matsp888 at yahoo.com
Mon Dec 28 15:45:20 CET 2015


On 12/28/2015 03:39 PM, Nicolas George wrote:
> L'octidi 8 nivôse, an CCXXIV, Michael Niedermayer a écrit :
>> for reference: (similar code prior to the patches)
>
> Indeed. Mats already pointed one of them. This would warrant some
> refactoring. At the very least:
>
>      ff_frame_new_palette(frame, palette);
>
> Then Ganesh will be able to add av_warn_unused_result on it ;) (And in the
> meantime, it could set AV_FRAME_FLAG_CORRUPT.=
>
> Regards,
>
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

A refactoring indeed sounds sensible to me.

Mats

-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/


More information about the ffmpeg-devel mailing list