[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