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

[Ganesh Ajjanagadde]

On Mon, Dec 28, 2015 at 6:39 AM, Nicolas George <george at nsup.org> 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.=

:D. It would be great if we had a way to track these; I may have
easily missed this message, given the huge volume of messages on this
thread. Noted in my todo list, thanks.

@Mats: this is a good example for why it is very helpful to reduce the
volume of email messages, and to think over them before sending to
avoid small additions/removals/clarifications.

>
> Regards,
>
> --
>   Nicolas George
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>