diff options
| author | Romain Guy <romainguy@google.com> | 2012-05-14 19:44:40 -0700 |
|---|---|---|
| committer | Romain Guy <romainguy@google.com> | 2012-05-14 19:44:40 -0700 |
| commit | e651cc6239616a202f6e96ebc2ed93b4b8b3627c (patch) | |
| tree | 7aa90f8cccc91555bd652d0a88a898a026cdc333 /core/java/android | |
| parent | 99a6ddd4cd8762654a575eb4ac3d0e5431d919b8 (diff) | |
| download | frameworks_base-e651cc6239616a202f6e96ebc2ed93b4b8b3627c.zip frameworks_base-e651cc6239616a202f6e96ebc2ed93b4b8b3627c.tar.gz frameworks_base-e651cc6239616a202f6e96ebc2ed93b4b8b3627c.tar.bz2 | |
Remove all Dalvik allocations from Cavnas.drawBitmap(int[], ...)
Change-Id: Ie28538a2104d21154fdc78a56525e7403f08287d
Diffstat (limited to 'core/java/android')
| -rw-r--r-- | core/java/android/view/GLES20Canvas.java | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/core/java/android/view/GLES20Canvas.java b/core/java/android/view/GLES20Canvas.java index 0bb5f9f..1631fb3 100644 --- a/core/java/android/view/GLES20Canvas.java +++ b/core/java/android/view/GLES20Canvas.java @@ -833,19 +833,40 @@ class GLES20Canvas extends HardwareCanvas { @Override public void drawBitmap(int[] colors, int offset, int stride, float x, float y, int width, int height, boolean hasAlpha, Paint paint) { + if (width < 0) { + throw new IllegalArgumentException("width must be >= 0"); + } + + if (height < 0) { + throw new IllegalArgumentException("height must be >= 0"); + } + + if (Math.abs(stride) < width) { + throw new IllegalArgumentException("abs(stride) must be >= width"); + } + + int lastScanline = offset + (height - 1) * stride; + int length = colors.length; + + if (offset < 0 || (offset + width > length) || lastScanline < 0 || + (lastScanline + width > length)) { + throw new ArrayIndexOutOfBoundsException(); + } + // Shaders are ignored when drawing bitmaps int modifier = paint != null ? setupColorFilter(paint) : MODIFIER_NONE; try { - final Bitmap.Config config = hasAlpha ? Bitmap.Config.ARGB_8888 : Bitmap.Config.RGB_565; - final Bitmap b = Bitmap.createBitmap(colors, offset, stride, width, height, config); final int nativePaint = paint == null ? 0 : paint.mNativePaint; - nDrawBitmap(mRenderer, b.mNativeBitmap, b.mBuffer, x, y, nativePaint); - b.recycle(); + nDrawBitmap(mRenderer, colors, offset, stride, x, y, + width, height, hasAlpha, nativePaint); } finally { if (modifier != MODIFIER_NONE) nResetModifiers(mRenderer, modifier); } } + private static native void nDrawBitmap(int renderer, int[] colors, int offset, int stride, + float x, float y, int width, int height, boolean hasAlpha, int nativePaint); + @Override public void drawBitmap(int[] colors, int offset, int stride, int x, int y, int width, int height, boolean hasAlpha, Paint paint) { |
