diff options
author | Glenn Kasten <gkasten@google.com> | 2012-01-10 10:46:34 -0800 |
---|---|---|
committer | Glenn Kasten <gkasten@google.com> | 2012-01-20 11:44:26 -0800 |
commit | 7fc9a6fdf146ded90b51c52f4a05d797294dcb85 (patch) | |
tree | 215cbb16f9babb95e8598123219abc9fed6156b9 /media/libmedia/mediametadataretriever.cpp | |
parent | 27a0d83f53ddbfd6449ab0e3c786742a2cd135a9 (diff) | |
download | frameworks_av-7fc9a6fdf146ded90b51c52f4a05d797294dcb85.zip frameworks_av-7fc9a6fdf146ded90b51c52f4a05d797294dcb85.tar.gz frameworks_av-7fc9a6fdf146ded90b51c52f4a05d797294dcb85.tar.bz2 |
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
Diffstat (limited to 'media/libmedia/mediametadataretriever.cpp')
-rw-r--r-- | media/libmedia/mediametadataretriever.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
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 @@ sp<MediaMetadataRetriever::DeathNotifier> MediaMetadataRetriever::sDeathNotifier const sp<IMediaPlayerService>& MediaMetadataRetriever::getService() { Mutex::Autolock lock(sServiceLock); - if (sService.get() == 0) { + if (sService == 0) { sp<IServiceManager> sm = defaultServiceManager(); sp<IBinder> binder; do { |