[FFmpeg-devel] [PATCH] lavfi/transpose: support more pix fmts

Paul B Mahol onemda at gmail.com
Wed Feb 13 11:50:39 CET 2013


On 2/13/13, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Feb 12, 2013 at 02:10:14PM +0000, Paul B Mahol wrote:
>> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> [...]
>> @@ -236,6 +246,10 @@ static int filter_frame(AVFilterLink *inlink,
>> AVFilterBufferRef *in)
>>                  for (x = 0; x < outw; x++)
>>                      *((uint32_t *)(dst + 4*x)) = *((uint32_t *)(src +
>> x*srclinesize + y*4));
>>                  break;
>> +            case 6:
>> +                for (x = 0; x < outw; x++)
>> +                    *((uint64_t *)(dst + 6*x)) = *((uint64_t *)(src +
>> x*srclinesize + y*6));
>> +                break;
>
> shouldnt this r/w 48bit instead of 64 ?
>

Probably, but how would one do that?
Calling 24 bit one twice?

> rest of the patch LGTM
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
>


More information about the ffmpeg-devel mailing list