Hi,

On Thu, Dec 15, 2011 at 4:46 PM, Janne Grunau <[email protected]>wrote:

> On 2011-12-15 16:18:26 -0800, Ronald S. Bultje wrote:
> >
> > On Thu, Dec 15, 2011 at 4:14 PM, Janne Grunau <[email protected]
> >wrote:
> >
> > > ---
> > >  libswscale/swscale_unscaled.c |   26 ++++++++++++++++++++++++++
> > >  1 files changed, 26 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/libswscale/swscale_unscaled.c
> b/libswscale/swscale_unscaled.c
> > > index 86bc5ac..8de4f75 100644
> > > --- a/libswscale/swscale_unscaled.c
> > > +++ b/libswscale/swscale_unscaled.c
> > > @@ -252,6 +252,25 @@ static void gray8aToPacked24(const uint8_t *src,
> > > uint8_t *dst, int num_pixels, c
> > >     }
> > >  }
> > >
> > > +static int rgb48_bswap(SwsContext *c, const uint8_t* src[], int
> > > srcStride[],
> > > +                       int srcSliceY, int srcSliceH,
> > > +                       uint8_t* dst[], int dstStride[])
> > > +{
> > > +    int i, j;
> > > +    uint16_t       *dstPtr =       (uint16_t *)dst[0];
> > > +    const uint16_t *srcPtr = (const uint16_t *)src[0];
> > > +
> > > +    for (i = 0; i < srcSliceH; i++) {
> > > +        for (j = 0; j < srcStride[0]>>1; j++) {
> > > +            dstPtr[j] = av_bswap16(srcPtr[j]);
> > > +        }
> > > +        srcPtr += srcStride[0]>>1;
> > > +        dstPtr += dstStride[0]>>1;
> > > +    }
> > >
> >
> > Don't you want to do int srcstr = srcStride[0] >> 1, dststr =
> dstStride[0]
> > >> 1; outside the loop?
>
> done.
>
> the same conversion works for GRAY16{BE,LE} too, so added that.
>
> Janne
> ---8<---
> ---
>  libswscale/swscale_unscaled.c |   30 ++++++++++++++++++++++++++++++
>  1 files changed, 30 insertions(+), 0 deletions(-)
>

LGTM.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to