Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Mats Peterson
On 02/27/2016 03:11 PM, Reimar Döffinger wrote: On Sat, Feb 27, 2016 at 02:58:39PM +0100, Mats Peterson wrote: And yes, QuickTime has a "default palette" for each bit depth that it can use, and no palette will be stored in the video sample description in that case. But that's only sensible to us

Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Reimar Döffinger
On Sat, Feb 27, 2016 at 02:58:39PM +0100, Mats Peterson wrote: > And yes, QuickTime has a "default palette" for each bit depth that it can > use, and no palette will be stored in the video sample description in that > case. But that's only sensible to use for 1 bpp black & white or 8 bpp > grayscal

Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Mats Peterson
On 02/27/2016 02:55 PM, Mats Peterson wrote: On 02/27/2016 02:44 PM, Reimar Döffinger wrote: On Sat, Feb 27, 2016 at 02:35:36PM +0100, Mats Peterson wrote: On 02/27/2016 02:03 PM, Mats Peterson wrote: Currently the only palettized pixel format in FFmpeg is AV_PIX_FMT_PAL8. In case there will b

Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Mats Peterson
On 02/27/2016 02:44 PM, Reimar Döffinger wrote: On Sat, Feb 27, 2016 at 02:35:36PM +0100, Mats Peterson wrote: On 02/27/2016 02:03 PM, Mats Peterson wrote: Currently the only palettized pixel format in FFmpeg is AV_PIX_FMT_PAL8. In case there will be other palettized formats in the future, I ha

Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Reimar Döffinger
On Sat, Feb 27, 2016 at 02:35:36PM +0100, Mats Peterson wrote: > On 02/27/2016 02:03 PM, Mats Peterson wrote: > >Currently the only palettized pixel format in FFmpeg is AV_PIX_FMT_PAL8. > >In case there will be other palettized formats in the future, I have > >"guarded" myself by using 1 << bits_pe

Re: [FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Mats Peterson
On 02/27/2016 02:03 PM, Mats Peterson wrote: Currently the only palettized pixel format in FFmpeg is AV_PIX_FMT_PAL8. In case there will be other palettized formats in the future, I have "guarded" myself by using 1 << bits_per_coded_sample in avienc.c and movenc.c for calculating the number of pa

[FFmpeg-devel] [PATCH] lavf/rawutils: Make ff_reshuffle_raw_rgb() somewhat more future-proof

2016-02-27 Thread Mats Peterson
Currently the only palettized pixel format in FFmpeg is AV_PIX_FMT_PAL8. In case there will be other palettized formats in the future, I have "guarded" myself by using 1 << bits_per_coded_sample in avienc.c and movenc.c for calculating the number of palette entries in packets containing a palet