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/IMediaDeathNotifier.cpp

diff --git a/media/libmedia/IMediaDeathNotifier.cpp b/media/libmedia/IMediaDeathNotifier.cpp
index 8525482..aeb35a5 100644
--- a/media/libmedia/IMediaDeathNotifier.cpp
+++ b/media/libmedia/IMediaDeathNotifier.cpp
@@ -36,7 +36,7 @@
 {
     ALOGV("getMediaPlayerService");
     Mutex::Autolock _l(sServiceLock);
-    if (sMediaPlayerService.get() == 0) {
+    if (sMediaPlayerService == 0) {
         sp<IServiceManager> sm = defaultServiceManager();
         sp<IBinder> binder;
         do {