API review - Image Ops (RT-17409)

[Kevin Rushforth]

+1

-- Kevin

Jim Graham wrote:
> By popular demand in internal reviews, I've added the following 
> methods to Image (with documentation analogous to the existing 
> get/setPixels methods):
>
>     public void getPixels(int x, int y, int w, int h,
>                           WritablePixelFormat<ByteBuffer> pixelformat,
>                           byte buffer[], int offset, int scanlineStride)
>     public void getPixels(int x, int y, int w, int h,
>                           WritablePixelFormat<IntBuffer> pixelformat,
>                           int buffer[], int offset, int scanlineStride)
>
>     public void setPixels(int x, int y, int w, int h,
>                           WritablePixelFormat<ByteBuffer> pixelformat,
>                           byte buffer[], int offset, int scanlineStride)
>     public void setPixels(int x, int y, int w, int h,
>                           WritablePixelFormat<IntBuffer> pixelformat,
>                           int buffer[], int offset, int scanlineStride)
>
> And a minor change to the existing get/setPixels methods to rearrange 
> their parameter lists to be more aligned with the above new methods 
> (note that buffer and PixelFormat have been swapped):
>
>     public <T extends Buffer>
>         void getPixels(int x, int y, int w, int h,
>                        WritablePixelFormat<T> pixelformat,
>                        T buffer, int scanlineStride)
>     public <T extends Buffer>
>         void setPixels(int x, int y, int w, int h,
>                        PixelFormat<T> pixelformat,
>                        T buffer, int scanlineStride)
>
>             ...jim