diff options
author | James Dong <jdong@google.com> | 2011-04-06 18:29:01 -0700 |
---|---|---|
committer | James Dong <jdong@google.com> | 2011-04-06 21:33:03 -0700 |
commit | 040e4a14ef9cafd5d08d9a180e6237fb79687995 (patch) | |
tree | c14582b7bb0b2596aeeca669ce2ec51dc4eaef3b /media/libmedia/IMediaPlayer.cpp | |
parent | c30268b9d118309a0514bcf280a03ee69f81403f (diff) | |
download | frameworks_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.cpp | 9 |
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); |