commit a95f495f75d0516ad48576500f64a3ffea037af8
author Dave Sparks <davidsparks@android.com> Fri Jul 10 18:13:36 2009 -0700
committer Dave Sparks <davidsparks@android.com> Fri Jul 10 18:13:36 2009 -0700
tree 51393ed5d0e6227181838d0b7835b924be5e2071
parent cf29e116129f38309c18ba14b9893b079a712289

Use SetByteArrayRegion to minimize copy overhead and JNI calls.

core/jni/android_hardware_Camera.cpp

diff --git a/core/jni/android_hardware_Camera.cpp b/core/jni/android_hardware_Camera.cpp
index b841076..2874be4 100644
--- a/core/jni/android_hardware_Camera.cpp
+++ b/core/jni/android_hardware_Camera.cpp
@@ -143,10 +143,7 @@
                 LOGE("Couldn't allocate byte array for JPEG data");
                 env->ExceptionClear();
             } else {
-                jbyte *bytes = static_cast<jbyte*>(env->GetPrimitiveArrayCritical(obj, NULL));
-                memcpy(bytes, data, size);
-                env->ReleasePrimitiveArrayCritical(obj, bytes, 0);
-
+                env->SetByteArrayRegion(obj, 0, size, data);
             }
         } else {
             LOGE("image heap is NULL");