diff options
author | Marco Nelissen <marcone@google.com> | 2013-02-05 09:03:22 -0800 |
---|---|---|
committer | Marco Nelissen <marcone@google.com> | 2013-02-05 09:03:22 -0800 |
commit | a77c496d4520f25b7b337d32ebd9681d8ea0f7bb (patch) | |
tree | 286d828a212c5b3040c072257cc49e0ab76865b0 /media/libstagefright/AwesomePlayer.cpp | |
parent | 3d6aa43575ae3342a93f88dd035f299e99554797 (diff) | |
download | frameworks_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.cpp | 1 |
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; } |