From edcd2cc572cb5823988efe0ed06feca5a937b1ff Mon Sep 17 00:00:00 2001
From: Mirko Bonadei <mbonadei@webrtc.org>
Date: Thu, 3 May 2018 10:35:23 +0200
Subject: [PATCH] Avoid allocation/copy by using GetFloatArrayRegion.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Bug: None
Change-Id: Ia049591f1d8d819d651ec8f359f318a7b9c12e43
Reviewed-on: https://webrtc-review.googlesource.com/74001
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23088}
---
 sdk/android/src/jni/videoframe.cc | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/sdk/android/src/jni/videoframe.cc b/sdk/android/src/jni/videoframe.cc
index ba9fca99dc..d85df4a1fb 100644
--- a/sdk/android/src/jni/videoframe.cc
+++ b/sdk/android/src/jni/videoframe.cc
@@ -125,11 +125,7 @@ int64_t GetJavaVideoFrameTimestampNs(JNIEnv* jni,
 
 Matrix::Matrix(JNIEnv* jni, const JavaRef<jfloatArray>& a) {
   RTC_CHECK_EQ(16, jni->GetArrayLength(a.obj()));
-  jfloat* ptr = jni->GetFloatArrayElements(a.obj(), nullptr);
-  for (int i = 0; i < 16; ++i) {
-    elem_[i] = ptr[i];
-  }
-  jni->ReleaseFloatArrayElements(a.obj(), ptr, 0);
+  jni->GetFloatArrayRegion(a.obj(), 0, 16, elem_);
 }
 
 ScopedJavaLocalRef<jfloatArray> Matrix::ToJava(JNIEnv* jni) const {