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

[Michael Niedermayer]

On Wed, Feb 13, 2013 at 10:50:39AM +0000, Paul B Mahol wrote:
> 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?

cleanest would probably be to add a AV_R/WL/B48

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130213/a196bfe5/attachment.asc>