diff options
| -rw-r--r-- | cmds/stagefright/codec.cpp | 4 | ||||
| -rw-r--r-- | cmds/stagefright/mediafilter.cpp | 4 | ||||
| -rw-r--r-- | media/libstagefright/ACodec.cpp | 5 | ||||
| -rw-r--r-- | media/libstagefright/MediaCodec.cpp | 2 | ||||
| -rw-r--r-- | media/libstagefright/codecs/flac/enc/Android.bp | 1 | ||||
| -rw-r--r-- | media/libstagefright/codecs/flac/enc/SoftFlacEncoder.cpp | 3 | ||||
| -rw-r--r-- | media/libstagefright/codecs/on2/enc/SoftVPXEncoder.cpp | 7 | ||||
| -rw-r--r-- | media/libstagefright/colorconversion/ColorConverter.cpp | 3 | ||||
| -rw-r--r-- | media/libstagefright/httplive/Android.bp | 4 | ||||
| -rw-r--r-- | media/libstagefright/httplive/PlaylistFetcher.cpp | 3 | ||||
| -rw-r--r-- | media/libstagefright/omx/Android.bp | 1 | ||||
| -rw-r--r-- | media/libstagefright/omx/OMXNodeInstance.cpp | 9 | ||||
| -rw-r--r-- | media/libstagefright/omx/OMXUtils.cpp | 2 | ||||
| -rw-r--r-- | media/libstagefright/omx/tests/Android.bp | 4 | ||||
| -rw-r--r-- | media/libstagefright/omx/tests/OMXHarness.cpp | 3 | ||||
| -rw-r--r-- | media/libstagefright/xmlparser/Android.bp | 4 | ||||
| -rw-r--r-- | media/libstagefright/xmlparser/MediaCodecsXmlParser.cpp | 3 |
17 files changed, 43 insertions, 19 deletions
diff --git a/cmds/stagefright/codec.cpp b/cmds/stagefright/codec.cpp index 6a58467d78..980d1d2c0b 100644 --- a/cmds/stagefright/codec.cpp +++ b/cmds/stagefright/codec.cpp @@ -366,13 +366,13 @@ int main(int argc, char **argv) { case 'T': { useTimestamp = true; + FALLTHROUGH_INTENDED; } - // fall through case 'R': { renderSurface = true; + FALLTHROUGH_INTENDED; } - // fall through case 'S': { useSurface = true; diff --git a/cmds/stagefright/mediafilter.cpp b/cmds/stagefright/mediafilter.cpp index f24d2ddd95..630de25999 100644 --- a/cmds/stagefright/mediafilter.cpp +++ b/cmds/stagefright/mediafilter.cpp @@ -706,13 +706,13 @@ int main(int argc, char **argv) { case 'T': { useTimestamp = true; + FALLTHROUGH_INTENDED; } - // fall through case 'R': { renderSurface = true; + FALLTHROUGH_INTENDED; } - // fall through case 'S': { useSurface = true; diff --git a/media/libstagefright/ACodec.cpp b/media/libstagefright/ACodec.cpp index 3526047800..68adf3ccb5 100644 --- a/media/libstagefright/ACodec.cpp +++ b/media/libstagefright/ACodec.cpp @@ -1619,7 +1619,7 @@ status_t ACodec::freeBuffer(OMX_U32 portIndex, size_t i) { if (portIndex == kPortIndexOutput && mNativeWindow != NULL) { (void)cancelBufferToNativeWindow(info); } - // fall through + FALLTHROUGH_INTENDED; case BufferInfo::OWNED_BY_NATIVE_WINDOW: err = mOMXNode->freeBuffer(portIndex, info->mBufferID); @@ -5019,6 +5019,7 @@ status_t ACodec::getPortFormat(OMX_U32 portIndex, sp<AMessage> ¬ify) { } } // Fall through to set up mime. + FALLTHROUGH_INTENDED; } default: @@ -7854,7 +7855,7 @@ bool ACodec::OutputPortSettingsChangedState::onMessageReceived( msg->setInt32("generation", mCodec->mStateGeneration); msg->post(3000000); } - // fall-through + FALLTHROUGH_INTENDED; } case kWhatResume: case kWhatSetParameters: diff --git a/media/libstagefright/MediaCodec.cpp b/media/libstagefright/MediaCodec.cpp index f91c5436b0..70064ea087 100644 --- a/media/libstagefright/MediaCodec.cpp +++ b/media/libstagefright/MediaCodec.cpp @@ -1829,8 +1829,8 @@ void MediaCodec::onMessageReceived(const sp<AMessage> &msg) { // the shutdown complete notification. If we // don't, we'll timeout and force release. sendErrorResponse = false; + FALLTHROUGH_INTENDED; } - // fall-thru case STOPPING: { if (mFlags & kFlagSawMediaServerDie) { diff --git a/media/libstagefright/codecs/flac/enc/Android.bp b/media/libstagefright/codecs/flac/enc/Android.bp index 46b974d158..73f0dacbc2 100644 --- a/media/libstagefright/codecs/flac/enc/Android.bp +++ b/media/libstagefright/codecs/flac/enc/Android.bp @@ -30,6 +30,7 @@ cc_library_shared { "liblog", ], + header_libs: ["libbase_headers"], static_libs: ["libFLAC"], name: "libstagefright_soft_flacenc", diff --git a/media/libstagefright/codecs/flac/enc/SoftFlacEncoder.cpp b/media/libstagefright/codecs/flac/enc/SoftFlacEncoder.cpp index fdc897570b..955f211f4f 100644 --- a/media/libstagefright/codecs/flac/enc/SoftFlacEncoder.cpp +++ b/media/libstagefright/codecs/flac/enc/SoftFlacEncoder.cpp @@ -16,6 +16,7 @@ //#define LOG_NDEBUG 0 #define LOG_TAG "SoftFlacEncoder" +#include <android-base/macros.h> #include <utils/Log.h> #include "SoftFlacEncoder.h" @@ -335,7 +336,7 @@ OMX_ERRORTYPE SoftFlacEncoder::internalSetParameter( } } - // fall through + FALLTHROUGH_INTENDED; } default: diff --git a/media/libstagefright/codecs/on2/enc/SoftVPXEncoder.cpp b/media/libstagefright/codecs/on2/enc/SoftVPXEncoder.cpp index f6257b1556..2dfba13a71 100644 --- a/media/libstagefright/codecs/on2/enc/SoftVPXEncoder.cpp +++ b/media/libstagefright/codecs/on2/enc/SoftVPXEncoder.cpp @@ -21,6 +21,7 @@ #include "SoftVP8Encoder.h" #include "SoftVP9Encoder.h" +#include <android-base/macros.h> #include <utils/Log.h> #include <utils/misc.h> @@ -557,7 +558,7 @@ vpx_enc_frame_flags_t SoftVPXEncoder::getEncodeFlags() { break; case kTemporalUpdateGoldenWithoutDependency: flags |= VP8_EFLAG_NO_REF_GF; - // Deliberately no break here. + FALLTHROUGH_INTENDED; case kTemporalUpdateGolden: flags |= VP8_EFLAG_NO_REF_ARF; flags |= VP8_EFLAG_NO_UPD_ARF; @@ -566,14 +567,14 @@ vpx_enc_frame_flags_t SoftVPXEncoder::getEncodeFlags() { case kTemporalUpdateAltrefWithoutDependency: flags |= VP8_EFLAG_NO_REF_ARF; flags |= VP8_EFLAG_NO_REF_GF; - // Deliberately no break here. + FALLTHROUGH_INTENDED; case kTemporalUpdateAltref: flags |= VP8_EFLAG_NO_UPD_GF; flags |= VP8_EFLAG_NO_UPD_LAST; break; case kTemporalUpdateNoneNoRefAltref: flags |= VP8_EFLAG_NO_REF_ARF; - // Deliberately no break here. + FALLTHROUGH_INTENDED; case kTemporalUpdateNone: flags |= VP8_EFLAG_NO_UPD_GF; flags |= VP8_EFLAG_NO_UPD_ARF; diff --git a/media/libstagefright/colorconversion/ColorConverter.cpp b/media/libstagefright/colorconversion/ColorConverter.cpp index 862cc63aa3..86bd9d628a 100644 --- a/media/libstagefright/colorconversion/ColorConverter.cpp +++ b/media/libstagefright/colorconversion/ColorConverter.cpp @@ -16,6 +16,7 @@ //#define LOG_NDEBUG 0 #define LOG_TAG "ColorConverter" +#include <android-base/macros.h> #include <utils/Log.h> #include <media/stagefright/foundation/ADebug.h> @@ -82,7 +83,7 @@ bool ColorConverter::isValid() const { if (mDstFormat == OMX_COLOR_FormatYUV444Y410) { return true; } - // fall-thru + FALLTHROUGH_INTENDED; case OMX_COLOR_FormatYUV420Planar: return mDstFormat == OMX_COLOR_Format16bitRGB565 || mDstFormat == OMX_COLOR_Format32BitRGBA8888 diff --git a/media/libstagefright/httplive/Android.bp b/media/libstagefright/httplive/Android.bp index 8a77401a5d..2f933a0395 100644 --- a/media/libstagefright/httplive/Android.bp +++ b/media/libstagefright/httplive/Android.bp @@ -46,6 +46,10 @@ cc_library { "android.hardware.cas.native@1.0", ], + header_libs: [ + "libbase_headers", + ], + static_libs: [ "libstagefright_id3", "libstagefright_metadatautils", diff --git a/media/libstagefright/httplive/PlaylistFetcher.cpp b/media/libstagefright/httplive/PlaylistFetcher.cpp index 9f39b5eb02..823f90e092 100644 --- a/media/libstagefright/httplive/PlaylistFetcher.cpp +++ b/media/libstagefright/httplive/PlaylistFetcher.cpp @@ -16,6 +16,7 @@ //#define LOG_NDEBUG 0 #define LOG_TAG "PlaylistFetcher" +#include <android-base/macros.h> #include <utils/Log.h> #include <utils/misc.h> @@ -267,7 +268,7 @@ int64_t PlaylistFetcher::delayUsToRefreshPlaylist() const { break; } - // fall through + FALLTHROUGH_INTENDED; } case FIRST_UNCHANGED_RELOAD_ATTEMPT: diff --git a/media/libstagefright/omx/Android.bp b/media/libstagefright/omx/Android.bp index 3e6942bc87..625004562d 100644 --- a/media/libstagefright/omx/Android.bp +++ b/media/libstagefright/omx/Android.bp @@ -29,6 +29,7 @@ cc_library_shared { ], header_libs: [ + "libbase_headers", "media_plugin_headers", ], diff --git a/media/libstagefright/omx/OMXNodeInstance.cpp b/media/libstagefright/omx/OMXNodeInstance.cpp index 32113c2621..99762b9999 100644 --- a/media/libstagefright/omx/OMXNodeInstance.cpp +++ b/media/libstagefright/omx/OMXNodeInstance.cpp @@ -16,6 +16,7 @@ //#define LOG_NDEBUG 0 #define LOG_TAG "OMXNodeInstance" +#include <android-base/macros.h> #include <utils/Log.h> #include <inttypes.h> @@ -459,7 +460,7 @@ status_t OMXNodeInstance::freeNode() { break; } - // fall through + FALLTHROUGH_INTENDED; } case OMX_StateIdle: @@ -486,7 +487,7 @@ status_t OMXNodeInstance::freeNode() { } CHECK_EQ(err, OMX_ErrorNone); - // fall through + FALLTHROUGH_INTENDED; } case OMX_StateLoaded: @@ -2196,8 +2197,8 @@ void OMXNodeInstance::onEvent( // bump internal-state debug level for 2 input and output frames Mutex::Autolock _l(mDebugLock); bumpDebugLevel_l(2 /* numInputBuffers */, 2 /* numOutputBuffers */); + FALLTHROUGH_INTENDED; } - // fall through default: arg2String = portString(arg2); } @@ -2208,7 +2209,7 @@ void OMXNodeInstance::onEvent( break; case OMX_EventPortSettingsChanged: arg2String = asString((OMX_INDEXEXTTYPE)arg2); - // fall through + FALLTHROUGH_INTENDED; default: arg1String = portString(arg1); } diff --git a/media/libstagefright/omx/OMXUtils.cpp b/media/libstagefright/omx/OMXUtils.cpp index f7b569dff6..9ed4a99aeb 100644 --- a/media/libstagefright/omx/OMXUtils.cpp +++ b/media/libstagefright/omx/OMXUtils.cpp @@ -19,6 +19,7 @@ #include <string.h> +#include <android-base/macros.h> #include <media/stagefright/omx/OMXUtils.h> #include <media/stagefright/foundation/ADebug.h> #include <media/stagefright/foundation/AUtils.h> @@ -273,6 +274,7 @@ bool DescribeDefaultColorFormat(DescribeColorFormat2Params ¶ms) { break; } else { // fall through as YV12 is used for YUV420Planar by some codecs + FALLTHROUGH_INTENDED; } case OMX_COLOR_FormatYUV420Planar: diff --git a/media/libstagefright/omx/tests/Android.bp b/media/libstagefright/omx/tests/Android.bp index 3b521ab1f7..ef36982806 100644 --- a/media/libstagefright/omx/tests/Android.bp +++ b/media/libstagefright/omx/tests/Android.bp @@ -27,6 +27,10 @@ cc_test { "frameworks/native/include/media/openmax", ], + header_libs: [ + "libbase_headers", + ], + cflags: [ "-Werror", "-Wall", diff --git a/media/libstagefright/omx/tests/OMXHarness.cpp b/media/libstagefright/omx/tests/OMXHarness.cpp index 5388ba7649..c2f1072469 100644 --- a/media/libstagefright/omx/tests/OMXHarness.cpp +++ b/media/libstagefright/omx/tests/OMXHarness.cpp @@ -17,6 +17,7 @@ //#define LOG_NDEBUG 0 #define LOG_TAG "OMXHarness" #include <inttypes.h> +#include <android-base/macros.h> #include <utils/Log.h> #include "OMXHarness.h" @@ -843,7 +844,7 @@ int main(int argc, char **argv) { case '?': fprintf(stderr, "\n"); - // fall through + FALLTHROUGH_INTENDED; case 'h': default: diff --git a/media/libstagefright/xmlparser/Android.bp b/media/libstagefright/xmlparser/Android.bp index a4fa342e01..5737ac2f7f 100644 --- a/media/libstagefright/xmlparser/Android.bp +++ b/media/libstagefright/xmlparser/Android.bp @@ -19,6 +19,10 @@ cc_library_shared { "libstagefright_omx_utils", ], + header_libs: [ + "libbase_headers", + ], + cflags: [ "-Werror", "-Wall", diff --git a/media/libstagefright/xmlparser/MediaCodecsXmlParser.cpp b/media/libstagefright/xmlparser/MediaCodecsXmlParser.cpp index ffd30eaf43..0832944115 100644 --- a/media/libstagefright/xmlparser/MediaCodecsXmlParser.cpp +++ b/media/libstagefright/xmlparser/MediaCodecsXmlParser.cpp @@ -19,6 +19,7 @@ #include <media/stagefright/xmlparser/MediaCodecsXmlParser.h> +#include <android-base/macros.h> #include <utils/Log.h> #include <media/stagefright/MediaErrors.h> #include <media/stagefright/omx/OMXUtils.h> @@ -340,7 +341,7 @@ void MediaCodecsXmlParser::startElementHandler( } } inType = false; - // fall through + FALLTHROUGH_INTENDED; case SECTION_DECODER_TYPE: case SECTION_ENCODER_TYPE: |
