commit 7fc9a6fdf146ded90b51c52f4a05d797294dcb85
author Glenn Kasten <gkasten@google.com> Tue Jan 10 10:46:34 2012 -0800
committer Glenn Kasten <gkasten@google.com> Fri Jan 20 11:44:26 2012 -0800
tree 215cbb16f9babb95e8598123219abc9fed6156b9
parent 27a0d83f53ddbfd6449ab0e3c786742a2cd135a9

Remove redundant get()

get() is almost always unnecessary, except in a LOG.
Also no need to check for != 0 before calling get().

Change-Id: Ib06e7a503f86cf102f09acc1ffb2ad085025516d

media/libmedia/mediametadataretriever.cpp

diff --git a/media/libmedia/mediametadataretriever.cpp b/media/libmedia/mediametadataretriever.cpp
index 88e269f..8d53357 100644
--- a/media/libmedia/mediametadataretriever.cpp
+++ b/media/libmedia/mediametadataretriever.cpp
@@ -35,7 +35,7 @@
 const sp<IMediaPlayerService>& MediaMetadataRetriever::getService()
 {
     Mutex::Autolock lock(sServiceLock);
-    if (sService.get() == 0) {
+    if (sService == 0) {
         sp<IServiceManager> sm = defaultServiceManager();
         sp<IBinder> binder;
         do {