Nicolas George wrote in <20141110140122.ga22...@phare.normalesup.org> >Thanks for the patch. See comments below.
O.K. I've finished to revise the patch.
Please see attached.
>The update for the documentation is missing.
Added short description to doc/decoders.texi.
>> + if (strncmp(p, "ifo:", 4) == 0) {
>
>This can be discussed, but I would prefer having a separate -ifo_file
>option.
I agree. Separated to -use_ifo_palette option.
Hendrik Leppkes wrote in
<ca+anqdwl+q+y9ve_vgmbnaks3w8lbtaazqetvjpeoqfsxwi...@mail.gmail.com>
>We have conversion macros for those, see yuv_a_to_rgba at the beginning of
>the same file.
>In fact, if you just read all the YCbCr stuff into a fixed-size buffer
>first, you might even be able to use the same function for the entire
>conversion.
Thanks! Those macros worked fine in my function.
--
TOYAMA Shin-ichi mailto:sh...@wmail.plala.or.jp
0001-New-option-for-obtaining-global-palette-from-.IFO-file.patch
Description: Binary data
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
