summaryrefslogtreecommitdiffstats
path: root/media/libstagefright/AwesomePlayer.cpp
diff options
context:
space:
mode:
authorMarco Nelissen <marcone@google.com>2013-02-05 09:03:22 -0800
committerMarco Nelissen <marcone@google.com>2013-02-05 09:03:22 -0800
commita77c496d4520f25b7b337d32ebd9681d8ea0f7bb (patch)
tree286d828a212c5b3040c072257cc49e0ab76865b0 /media/libstagefright/AwesomePlayer.cpp
parent3d6aa43575ae3342a93f88dd035f299e99554797 (diff)
downloadframeworks_av-a77c496d4520f25b7b337d32ebd9681d8ea0f7bb.zip
frameworks_av-a77c496d4520f25b7b337d32ebd9681d8ea0f7bb.tar.gz
frameworks_av-a77c496d4520f25b7b337d32ebd9681d8ea0f7bb.tar.bz2
Return error code if setting scaling mode fails
The return code from native_window_set_scaling_mode() was ignored. Looking at the code review comments that introduced this code, it seems like the intention was to return the error code in that case. Change-Id: I9592cc378f0a0b960d37178aa0525fc17e8734ba
Diffstat (limited to 'media/libstagefright/AwesomePlayer.cpp')
-rw-r--r--media/libstagefright/AwesomePlayer.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/media/libstagefright/AwesomePlayer.cpp b/media/libstagefright/AwesomePlayer.cpp
index 23ce088..0f4d866 100644
--- a/media/libstagefright/AwesomePlayer.cpp
+++ b/media/libstagefright/AwesomePlayer.cpp
@@ -2511,6 +2511,7 @@ status_t AwesomePlayer::setVideoScalingMode_l(int32_t mode) {
if (err != OK) {
ALOGW("Failed to set scaling mode: %d", err);
}
+ return err;
}
return OK;
}