diff options
author | Ray Essick <essick@google.com> | 2019-01-28 20:34:42 -0800 |
---|---|---|
committer | Ray Essick <essick@google.com> | 2019-01-28 20:34:42 -0800 |
commit | e9b9c98a573d000abecc6d5a1718a78c06fff551 (patch) | |
tree | 2f2abe6e64b42ed45ab83cfc2f9a4374f5088733 /media/jni/android_media_MediaExtractor.cpp | |
parent | 5e55e809a722f1540630062d356c504169b3a688 (diff) |
use MediaAnalyticsItem::create() instead of new
Bug: 123543489
Test: boot/logcat, select CtsMediaTests
Diffstat (limited to 'media/jni/android_media_MediaExtractor.cpp')
-rw-r--r-- | media/jni/android_media_MediaExtractor.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/media/jni/android_media_MediaExtractor.cpp b/media/jni/android_media_MediaExtractor.cpp index f3442f4b4d33..c6b171bdd6d4 100644 --- a/media/jni/android_media_MediaExtractor.cpp +++ b/media/jni/android_media_MediaExtractor.cpp @@ -913,13 +913,9 @@ android_media_MediaExtractor_native_getMetrics(JNIEnv * env, jobject thiz) } // build and return the Bundle - MediaAnalyticsItem *item = new MediaAnalyticsItem; + std::unique_ptr<MediaAnalyticsItem> item(MediaAnalyticsItem::create()); item->readFromParcel(reply); - jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item, NULL); - - // housekeeping - delete item; - item = NULL; + jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item.get(), NULL); return mybundle; } |