summaryrefslogtreecommitdiffstats
path: root/media/libmedia/IMediaPlayer.cpp
diff options
context:
space:
mode:
authorJames Dong <jdong@google.com>2011-04-06 18:29:01 -0700
committerJames Dong <jdong@google.com>2011-04-06 21:33:03 -0700
commit040e4a14ef9cafd5d08d9a180e6237fb79687995 (patch)
treec14582b7bb0b2596aeeca669ce2ec51dc4eaef3b /media/libmedia/IMediaPlayer.cpp
parentc30268b9d118309a0514bcf280a03ee69f81403f (diff)
downloadframeworks_av-040e4a14ef9cafd5d08d9a180e6237fb79687995.zip
frameworks_av-040e4a14ef9cafd5d08d9a180e6237fb79687995.tar.gz
frameworks_av-040e4a14ef9cafd5d08d9a180e6237fb79687995.tar.bz2
Return status code from invoke()
Change-Id: Ie64b418d2a1f44e07942399c285b460c68f8e14f related-to-bug: 4199781
Diffstat (limited to 'media/libmedia/IMediaPlayer.cpp')
-rw-r--r--media/libmedia/IMediaPlayer.cpp9
1 files changed, 5 insertions, 4 deletions
diff --git a/media/libmedia/IMediaPlayer.cpp b/media/libmedia/IMediaPlayer.cpp
index 2399216..8885bd5 100644
--- a/media/libmedia/IMediaPlayer.cpp
+++ b/media/libmedia/IMediaPlayer.cpp
@@ -192,8 +192,9 @@ public:
}
status_t invoke(const Parcel& request, Parcel *reply)
- { // Avoid doing any extra copy. The interface descriptor should
- // have been set by MediaPlayer.java.
+ {
+ // Avoid doing any extra copy. The interface descriptor should
+ // have been set by MediaPlayer.java.
return remote()->transact(INVOKE, request, reply);
}
@@ -334,8 +335,8 @@ status_t BnMediaPlayer::onTransact(
} break;
case INVOKE: {
CHECK_INTERFACE(IMediaPlayer, data, reply);
- invoke(data, reply);
- return NO_ERROR;
+ status_t result = invoke(data, reply);
+ return result;
} break;
case SET_METADATA_FILTER: {
CHECK_INTERFACE(IMediaPlayer, data, reply);