From 4edf1d10d44127484efe57f29c0204a29ad7e6ea Mon Sep 17 00:00:00 2001 From: James Dong Date: Wed, 6 Apr 2011 18:29:01 -0700 Subject: [PATCH] Return status code from invoke() Change-Id: Ie64b418d2a1f44e07942399c285b460c68f8e14f related-to-bug: 4199781 --- media/libmedia/IMediaPlayer.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/media/libmedia/IMediaPlayer.cpp b/media/libmedia/IMediaPlayer.cpp index 2399216b49ed9..8885bd5914e94 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);