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 | 200d71c850e308d4153ea2ea2ebc0004bcae7808 (patch) | |
tree | df348c603a7ce5cc4eccc3106b8ead6f0529ea8a /media | |
parent | 4f5454ecf49b1021cdd492f7764f013b39676bc6 (diff) | |
download | frameworks_av-200d71c850e308d4153ea2ea2ebc0004bcae7808.zip frameworks_av-200d71c850e308d4153ea2ea2ebc0004bcae7808.tar.gz frameworks_av-200d71c850e308d4153ea2ea2ebc0004bcae7808.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')
-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; } |