From d71a231c493999bc885eabcf4c856978b9ecff97 Mon Sep 17 00:00:00 2001 From: Urvang Joshi Date: Thu, 14 Jul 2016 12:33:48 -0700 Subject: [PATCH] Add compiler warning flag -Wextra and fix related warnings. Note: some of these warnings are enabled by a combination of -Wunused (added earlier) and -Wextra. Cherry-picked from aomedia/master: 4790a69 Change-Id: I322a1366bd4fd6c0dec9e758c2d5e88e003b1cbf --- aom_dsp/arm/aom_convolve8_avg_neon.c | 8 +++++++ aom_dsp/arm/aom_convolve8_neon.c | 8 +++++++ aom_dsp/x86/convolve.h | 4 ++++ aomenc.c | 5 ++--- configure | 4 ++++ examples/aom_cx_set_ref.c | 32 ++++++++++++++++------------ examples/lossless_encoder.c | 6 +++++- examples/simple_encoder.c | 6 +++++- test/av1_convolve_optimz_test.cc | 21 +++++++++--------- test/avg_test.cc | 9 ++++---- test/codec_factory.h | 9 ++++++++ test/dct16x16_test.cc | 6 +++--- test/decode_test_driver.cc | 2 +- test/decode_test_driver.h | 1 - test/encode_test_driver.cc | 4 ++-- test/encode_test_driver.h | 1 - test/encoder_parms_get_to_decoder.cc | 1 - test/error_resilience_test.cc | 3 +-- test/frame_size_tests.cc | 1 - webmenc.cc | 1 - webmenc.h | 1 - 21 files changed, 85 insertions(+), 48 deletions(-) diff --git a/aom_dsp/arm/aom_convolve8_avg_neon.c b/aom_dsp/arm/aom_convolve8_avg_neon.c index 7dc936d6e..09429d6d2 100644 --- a/aom_dsp/arm/aom_convolve8_avg_neon.c +++ b/aom_dsp/arm/aom_convolve8_avg_neon.c @@ -65,6 +65,10 @@ void aom_convolve8_avg_horiz_neon(const uint8_t *src, ptrdiff_t src_stride, assert(x_step_q4 == 16); + (void)x_step_q4; + (void)y_step_q4; + (void)filter_y; + q0s16 = vld1q_s16(filter_x); src -= 3; // adjust for taps @@ -241,6 +245,10 @@ void aom_convolve8_avg_vert_neon(const uint8_t *src, ptrdiff_t src_stride, assert(y_step_q4 == 16); + (void)x_step_q4; + (void)y_step_q4; + (void)filter_x; + src -= src_stride * 3; q0s16 = vld1q_s16(filter_y); for (; w > 0; w -= 4, src += 4, dst += 4) { // loop_vert_h diff --git a/aom_dsp/arm/aom_convolve8_neon.c b/aom_dsp/arm/aom_convolve8_neon.c index ed0df6dd2..8ebffb5f9 100644 --- a/aom_dsp/arm/aom_convolve8_neon.c +++ b/aom_dsp/arm/aom_convolve8_neon.c @@ -65,6 +65,10 @@ void aom_convolve8_horiz_neon(const uint8_t *src, ptrdiff_t src_stride, assert(x_step_q4 == 16); + (void)x_step_q4; + (void)y_step_q4; + (void)filter_y; + q0s16 = vld1q_s16(filter_x); src -= 3; // adjust for taps @@ -225,6 +229,10 @@ void aom_convolve8_vert_neon(const uint8_t *src, ptrdiff_t src_stride, assert(y_step_q4 == 16); + (void)x_step_q4; + (void)y_step_q4; + (void)filter_x; + src -= src_stride * 3; q0s16 = vld1q_s16(filter_y); for (; w > 0; w -= 4, src += 4, dst += 4) { // loop_vert_h diff --git a/aom_dsp/x86/convolve.h b/aom_dsp/x86/convolve.h index ffaed0212..87ff34be1 100644 --- a/aom_dsp/x86/convolve.h +++ b/aom_dsp/x86/convolve.h @@ -27,6 +27,10 @@ typedef void filter8_1dfunction(const uint8_t *src_ptr, ptrdiff_t src_pitch, const uint8_t *src, ptrdiff_t src_stride, uint8_t *dst, \ ptrdiff_t dst_stride, const int16_t *filter_x, int x_step_q4, \ const int16_t *filter_y, int y_step_q4, int w, int h) { \ + (void)filter_x; \ + (void)x_step_q4; \ + (void)filter_y; \ + (void)y_step_q4; \ assert(filter[3] != 128); \ assert(step_q4 == 16); \ if (filter[0] | filter[1] | filter[2]) { \ diff --git a/aomenc.c b/aomenc.c index 5b8632b82..7fa08af52 100644 --- a/aomenc.c +++ b/aomenc.c @@ -1412,9 +1412,8 @@ static void open_output_file(struct stream_state *stream, #if CONFIG_WEBM_IO if (stream->config.write_webm) { stream->webm_ctx.stream = stream->file; - write_webm_file_header(&stream->webm_ctx, cfg, &global->framerate, - stream->config.stereo_fmt, global->codec->fourcc, - pixel_aspect_ratio); + write_webm_file_header(&stream->webm_ctx, cfg, stream->config.stereo_fmt, + global->codec->fourcc, pixel_aspect_ratio); } #else (void)pixel_aspect_ratio; diff --git a/configure b/configure index 38a0bb698..86a74a9a0 100755 --- a/configure +++ b/configure @@ -617,6 +617,10 @@ process_toolchain() { check_add_cflags -Wuninitialized check_add_cflags -Wunused check_add_cflags -Wsign-compare + # Enabling the following warning (in combination with -Wunused above) + # for C++ generates errors in third_party code including googletest and + # libyuv. So enable it only for C code. + check_cflags "-Wextra" && add_cflags_only "-Wextra" # Enabling the following warning for C++ generates some useless warnings # about some function parameters shadowing class member function names. # So, only enable this warning for C code. diff --git a/examples/aom_cx_set_ref.c b/examples/aom_cx_set_ref.c index fdb9739bc..6beb4fb06 100644 --- a/examples/aom_cx_set_ref.c +++ b/examples/aom_cx_set_ref.c @@ -191,8 +191,7 @@ static void find_mismatch(const aom_image_t *const img1, } static void testing_decode(aom_codec_ctx_t *encoder, aom_codec_ctx_t *decoder, - aom_codec_enc_cfg_t *cfg, unsigned int frame_out, - int *mismatch_seen) { + unsigned int frame_out, int *mismatch_seen) { aom_image_t enc_img, dec_img; struct av1_ref_frame ref_enc, ref_dec; @@ -226,11 +225,10 @@ static void testing_decode(aom_codec_ctx_t *encoder, aom_codec_ctx_t *decoder, aom_img_free(&dec_img); } -static int encode_frame(aom_codec_ctx_t *ecodec, aom_codec_enc_cfg_t *cfg, - aom_image_t *img, unsigned int frame_in, - AvxVideoWriter *writer, int test_decode, - aom_codec_ctx_t *dcodec, unsigned int *frame_out, - int *mismatch_seen) { +static int encode_frame(aom_codec_ctx_t *ecodec, aom_image_t *img, + unsigned int frame_in, AvxVideoWriter *writer, + int test_decode, aom_codec_ctx_t *dcodec, + unsigned int *frame_out, int *mismatch_seen) { int got_pkts = 0; aom_codec_iter_t iter = NULL; const aom_codec_cx_pkt_t *pkt = NULL; @@ -271,7 +269,7 @@ static int encode_frame(aom_codec_ctx_t *ecodec, aom_codec_enc_cfg_t *cfg, // Mismatch checking if (got_data && test_decode) { - testing_decode(ecodec, dcodec, cfg, *frame_out, mismatch_seen); + testing_decode(ecodec, dcodec, *frame_out, mismatch_seen); } return got_pkts; @@ -280,12 +278,12 @@ static int encode_frame(aom_codec_ctx_t *ecodec, aom_codec_enc_cfg_t *cfg, int main(int argc, char **argv) { FILE *infile = NULL; // Encoder - aom_codec_ctx_t ecodec = { 0 }; - aom_codec_enc_cfg_t cfg = { 0 }; + aom_codec_ctx_t ecodec; + aom_codec_enc_cfg_t cfg; unsigned int frame_in = 0; aom_image_t raw; aom_codec_err_t res; - AvxVideoInfo info = { 0 }; + AvxVideoInfo info; AvxVideoWriter *writer = NULL; const AvxInterface *encoder = NULL; @@ -311,6 +309,12 @@ int main(int argc, char **argv) { unsigned int limit = 0; exec_name = argv[0]; + // Clear explicitly, as simply assigning "{ 0 }" generates + // "missing-field-initializers" warning in some compilers. + memset(&ecodec, 0, sizeof(ecodec)); + memset(&cfg, 0, sizeof(cfg)); + memset(&info, 0, sizeof(info)); + if (argc < 7) die("Invalid number of arguments"); codec_arg = argv[1]; @@ -404,7 +408,7 @@ int main(int argc, char **argv) { } } - encode_frame(&ecodec, &cfg, &raw, frame_in, writer, test_decode, &dcodec, + encode_frame(&ecodec, &raw, frame_in, writer, test_decode, &dcodec, &frame_out, &mismatch_seen); frame_in++; if (mismatch_seen) break; @@ -412,8 +416,8 @@ int main(int argc, char **argv) { // Flush encoder. if (!mismatch_seen) - while (encode_frame(&ecodec, &cfg, NULL, frame_in, writer, test_decode, - &dcodec, &frame_out, &mismatch_seen)) { + while (encode_frame(&ecodec, NULL, frame_in, writer, test_decode, &dcodec, + &frame_out, &mismatch_seen)) { } printf("\n"); diff --git a/examples/lossless_encoder.c b/examples/lossless_encoder.c index 069e35e32..1abeb2721 100644 --- a/examples/lossless_encoder.c +++ b/examples/lossless_encoder.c @@ -63,13 +63,17 @@ int main(int argc, char **argv) { int frame_count = 0; aom_image_t raw; aom_codec_err_t res; - AvxVideoInfo info = { 0 }; + AvxVideoInfo info; AvxVideoWriter *writer = NULL; const AvxInterface *encoder = NULL; const int fps = 30; exec_name = argv[0]; + // Clear explicitly, as simply assigning "{ 0 }" generates + // "missing-field-initializers" warning in some compilers. + memset(&info, 0, sizeof(info)); + if (argc < 5) die("Invalid number of arguments"); encoder = get_aom_encoder_by_name("av1"); diff --git a/examples/simple_encoder.c b/examples/simple_encoder.c index 418757def..1d2b51ed2 100644 --- a/examples/simple_encoder.c +++ b/examples/simple_encoder.c @@ -151,7 +151,7 @@ int main(int argc, char **argv) { int frame_count = 0; aom_image_t raw; aom_codec_err_t res; - AvxVideoInfo info = { 0 }; + AvxVideoInfo info; AvxVideoWriter *writer = NULL; const AvxInterface *encoder = NULL; const int fps = 30; @@ -168,6 +168,10 @@ int main(int argc, char **argv) { exec_name = argv[0]; + // Clear explicitly, as simply assigning "{ 0 }" generates + // "missing-field-initializers" warning in some compilers. + memset(&info, 0, sizeof(info)); + if (argc != 9) die("Invalid number of arguments"); codec_arg = argv[1]; diff --git a/test/av1_convolve_optimz_test.cc b/test/av1_convolve_optimz_test.cc index b83ae94b2..b891e9988 100644 --- a/test/av1_convolve_optimz_test.cc +++ b/test/av1_convolve_optimz_test.cc @@ -54,7 +54,6 @@ const size_t maxHeight = 256; const size_t maxBlockSize = maxWidth * maxHeight; const int horizOffset = 32; const int vertiOffset = 32; -const size_t testMaxBlk = 128; const int stride = 128; const int x_step_q4 = 16; @@ -90,7 +89,7 @@ class AV1ConvolveOptimzTest : public ::testing::TestWithParam { void RunVertFilterBitExactCheck(); private: - void PrepFilterBuffer(int w, int h); + void PrepFilterBuffer(); void DiffFilterBuffer(); conv_filter_t conv_horiz_; conv_filter_t conv_vert_; @@ -106,7 +105,7 @@ class AV1ConvolveOptimzTest : public ::testing::TestWithParam { int avg_; }; -void AV1ConvolveOptimzTest::PrepFilterBuffer(int w, int h) { +void AV1ConvolveOptimzTest::PrepFilterBuffer() { int r, c; ACMRandom rnd(ACMRandom::DeterministicSeed()); @@ -150,7 +149,7 @@ void AV1ConvolveOptimzTest::DiffFilterBuffer() { } void AV1ConvolveOptimzTest::RunHorizFilterBitExactCheck() { - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); InterpFilterParams filter_params = av1_get_interp_filter_params(filter_); @@ -167,7 +166,7 @@ void AV1ConvolveOptimzTest::RunHorizFilterBitExactCheck() { // and test again. int intermediate_height = (((height_ - 1) * 16 + subpel_) >> SUBPEL_BITS) + filter_params.taps; - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); av1_convolve_horiz_c(src_ref_, stride, dst_ref_, stride, width_, intermediate_height, filter_params, subpel_, x_step_q4, @@ -180,7 +179,7 @@ void AV1ConvolveOptimzTest::RunHorizFilterBitExactCheck() { } void AV1ConvolveOptimzTest::RunVertFilterBitExactCheck() { - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); InterpFilterParams filter_params = av1_get_interp_filter_params(filter_); @@ -266,7 +265,7 @@ class AV1HbdConvolveOptimzTest : public TestWithHbdConvParams { void RunVertFilterBitExactCheck(); private: - void PrepFilterBuffer(int w, int h); + void PrepFilterBuffer(); void DiffFilterBuffer(); hbd_conv_filter_t conv_horiz_; hbd_conv_filter_t conv_vert_; @@ -283,7 +282,7 @@ class AV1HbdConvolveOptimzTest : public TestWithHbdConvParams { int bit_depth_; }; -void AV1HbdConvolveOptimzTest::PrepFilterBuffer(int w, int h) { +void AV1HbdConvolveOptimzTest::PrepFilterBuffer() { int r, c; ACMRandom rnd(ACMRandom::DeterministicSeed()); @@ -326,7 +325,7 @@ void AV1HbdConvolveOptimzTest::DiffFilterBuffer() { } void AV1HbdConvolveOptimzTest::RunHorizFilterBitExactCheck() { - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); InterpFilterParams filter_params = av1_get_interp_filter_params(filter_); @@ -344,7 +343,7 @@ void AV1HbdConvolveOptimzTest::RunHorizFilterBitExactCheck() { // and test again. int intermediate_height = (((height_ - 1) * 16 + subpel_) >> SUBPEL_BITS) + filter_params.taps; - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); av1_highbd_convolve_horiz_c(src_, stride, dst_ref_, stride, width_, intermediate_height, filter_params, subpel_, @@ -357,7 +356,7 @@ void AV1HbdConvolveOptimzTest::RunHorizFilterBitExactCheck() { } void AV1HbdConvolveOptimzTest::RunVertFilterBitExactCheck() { - PrepFilterBuffer(testMaxBlk, testMaxBlk); + PrepFilterBuffer(); InterpFilterParams filter_params = av1_get_interp_filter_params(filter_); diff --git a/test/avg_test.cc b/test/avg_test.cc index cf6f89caa..b7a707d4e 100644 --- a/test/avg_test.cc +++ b/test/avg_test.cc @@ -54,14 +54,14 @@ class AverageTestBase : public ::testing::Test { } // Sum Pixels - unsigned int ReferenceAverage8x8(const uint8_t *source, int pitch) { + static unsigned int ReferenceAverage8x8(const uint8_t *source, int pitch) { unsigned int average = 0; for (int h = 0; h < 8; ++h) for (int w = 0; w < 8; ++w) average += source[h * pitch + w]; return ((average + 32) >> 6); } - unsigned int ReferenceAverage4x4(const uint8_t *source, int pitch) { + static unsigned int ReferenceAverage4x4(const uint8_t *source, int pitch) { unsigned int average = 0; for (int h = 0; h < 4; ++h) for (int w = 0; w < 4; ++w) average += source[h * pitch + w]; @@ -97,11 +97,12 @@ class AverageTest : public AverageTestBase, protected: void CheckAverages() { + const int block_size = GET_PARAM(3); unsigned int expected = 0; - if (GET_PARAM(3) == 8) { + if (block_size == 8) { expected = ReferenceAverage8x8(source_data_ + GET_PARAM(2), source_stride_); - } else if (GET_PARAM(3) == 4) { + } else if (block_size == 4) { expected = ReferenceAverage4x4(source_data_ + GET_PARAM(2), source_stride_); } diff --git a/test/codec_factory.h b/test/codec_factory.h index c92d5c1cf..b645102de 100644 --- a/test/codec_factory.h +++ b/test/codec_factory.h @@ -123,6 +123,9 @@ class AV1CodecFactory : public CodecFactory { #if CONFIG_AV1_DECODER return new AV1Decoder(cfg, flags, deadline); #else + (void)cfg; + (void)flags; + (void)deadline; return NULL; #endif } @@ -134,6 +137,10 @@ class AV1CodecFactory : public CodecFactory { #if CONFIG_AV1_ENCODER return new AV1Encoder(cfg, deadline, init_flags, stats); #else + (void)cfg; + (void)deadline; + (void)init_flags; + (void)stats; return NULL; #endif } @@ -143,6 +150,8 @@ class AV1CodecFactory : public CodecFactory { #if CONFIG_AV1_ENCODER return aom_codec_enc_config_default(&aom_codec_av1_cx_algo, cfg, usage); #else + (void)cfg; + (void)usage; return AOM_CODEC_INCAPABLE; #endif } diff --git a/test/dct16x16_test.cc b/test/dct16x16_test.cc index e73daa553..981195595 100644 --- a/test/dct16x16_test.cc +++ b/test/dct16x16_test.cc @@ -264,12 +264,12 @@ void idct16x16_12(const tran_low_t *in, uint8_t *out, int stride) { } void idct16x16_10_ref(const tran_low_t *in, uint8_t *out, int stride, - int tx_type) { + int /*tx_type*/) { idct16x16_10(in, out, stride); } void idct16x16_12_ref(const tran_low_t *in, uint8_t *out, int stride, - int tx_type) { + int /*tx_type*/) { idct16x16_12(in, out, stride); } @@ -727,7 +727,7 @@ class InvTrans16x16DCT : public Trans16x16TestBase, virtual void TearDown() { libaom_test::ClearSystemState(); } protected: - void RunFwdTxfm(int16_t *in, tran_low_t *out, int stride) {} + void RunFwdTxfm(int16_t * /*in*/, tran_low_t * /*out*/, int /*stride*/) {} void RunInvTxfm(tran_low_t *out, uint8_t *dst, int stride) { inv_txfm_(out, dst, stride); } diff --git a/test/decode_test_driver.cc b/test/decode_test_driver.cc index 95a0eb543..7adb9d616 100644 --- a/test/decode_test_driver.cc +++ b/test/decode_test_driver.cc @@ -92,7 +92,7 @@ void DecoderTest::RunLoop(CompressedVideoSource *video, aom_codec_err_t res_dec = decoder->DecodeFrame(video->cxdata(), video->frame_size()); - if (!HandleDecodeResult(res_dec, *video, decoder)) break; + if (!HandleDecodeResult(res_dec, decoder)) break; } else { // Signal end of the file to the decoder. const aom_codec_err_t res_dec = decoder->DecodeFrame(NULL, 0); diff --git a/test/decode_test_driver.h b/test/decode_test_driver.h index aabca4014..b8f8d1a61 100644 --- a/test/decode_test_driver.h +++ b/test/decode_test_driver.h @@ -141,7 +141,6 @@ class DecoderTest { // Hook to be called to handle decode result. Return true to continue. virtual bool HandleDecodeResult(const aom_codec_err_t res_dec, - const CompressedVideoSource & /*video*/, Decoder *decoder) { EXPECT_EQ(AOM_CODEC_OK, res_dec) << decoder->DecodeError(); return AOM_CODEC_OK == res_dec; diff --git a/test/encode_test_driver.cc b/test/encode_test_driver.cc index c1a0cb77e..092e66958 100644 --- a/test/encode_test_driver.cc +++ b/test/encode_test_driver.cc @@ -275,7 +275,7 @@ void EncoderTest::RunLoop(VideoSource *video) { aom_codec_err_t res_dec = decoder->DecodeFrame( (const uint8_t *)pkt->data.frame.buf, pkt->data.frame.sz); - if (!HandleDecodeResult(res_dec, *video, decoder.get())) break; + if (!HandleDecodeResult(res_dec, decoder.get())) break; has_dxdata = true; } @@ -293,7 +293,7 @@ void EncoderTest::RunLoop(VideoSource *video) { // Flush the decoder when there are no more fragments. if ((init_flags_ & AOM_CODEC_USE_OUTPUT_PARTITION) && has_dxdata) { const aom_codec_err_t res_dec = decoder->DecodeFrame(NULL, 0); - if (!HandleDecodeResult(res_dec, *video, decoder.get())) break; + if (!HandleDecodeResult(res_dec, decoder.get())) break; } if (has_dxdata && has_cxdata) { diff --git a/test/encode_test_driver.h b/test/encode_test_driver.h index 11c387af3..45a080e77 100644 --- a/test/encode_test_driver.h +++ b/test/encode_test_driver.h @@ -228,7 +228,6 @@ class EncoderTest { // Hook to be called to handle decode result. Return true to continue. virtual bool HandleDecodeResult(const aom_codec_err_t res_dec, - const VideoSource & /*video*/, Decoder *decoder) { EXPECT_EQ(AOM_CODEC_OK, res_dec) << decoder->DecodeError(); return AOM_CODEC_OK == res_dec; diff --git a/test/encoder_parms_get_to_decoder.cc b/test/encoder_parms_get_to_decoder.cc index 640e12f08..52d68b1f4 100644 --- a/test/encoder_parms_get_to_decoder.cc +++ b/test/encoder_parms_get_to_decoder.cc @@ -94,7 +94,6 @@ class AvxEncoderParmsGetToDecoder } virtual bool HandleDecodeResult(const aom_codec_err_t res_dec, - const libaom_test::VideoSource & /*video*/, libaom_test::Decoder *decoder) { aom_codec_ctx_t *const av1_decoder = decoder->GetDecoder(); aom_codec_alg_priv_t *const priv = diff --git a/test/error_resilience_test.cc b/test/error_resilience_test.cc index 951c47f29..07b60399e 100644 --- a/test/error_resilience_test.cc +++ b/test/error_resilience_test.cc @@ -55,8 +55,7 @@ class ErrorResilienceTestLarge nframes_++; } - virtual void PreEncodeFrameHook(libaom_test::VideoSource *video, - ::libaom_test::Encoder * /*encoder*/) { + virtual void PreEncodeFrameHook(libaom_test::VideoSource *video) { frame_flags_ &= ~(AOM_EFLAG_NO_UPD_LAST | AOM_EFLAG_NO_UPD_GF | AOM_EFLAG_NO_UPD_ARF); if (droppable_nframes_ > 0 && diff --git a/test/frame_size_tests.cc b/test/frame_size_tests.cc index f1fad7076..25b87180b 100644 --- a/test/frame_size_tests.cc +++ b/test/frame_size_tests.cc @@ -28,7 +28,6 @@ class AV1FrameSizeTestsLarge : public ::libaom_test::EncoderTest, } virtual bool HandleDecodeResult(const aom_codec_err_t res_dec, - const libaom_test::VideoSource & /*video*/, libaom_test::Decoder *decoder) { EXPECT_EQ(expected_res_, res_dec) << decoder->DecodeError(); return !::testing::Test::HasFailure(); diff --git a/webmenc.cc b/webmenc.cc index f78f02700..e3d209a27 100644 --- a/webmenc.cc +++ b/webmenc.cc @@ -24,7 +24,6 @@ const int kVideoTrackNumber = 1; void write_webm_file_header(struct WebmOutputContext *webm_ctx, const aom_codec_enc_cfg_t *cfg, - const struct aom_rational *fps, stereo_format_t stereo_fmt, unsigned int fourcc, const struct AvxRational *par) { mkvmuxer::MkvWriter *const writer = new mkvmuxer::MkvWriter(webm_ctx->stream); diff --git a/webmenc.h b/webmenc.h index 90211ffcc..74387fb8d 100644 --- a/webmenc.h +++ b/webmenc.h @@ -40,7 +40,6 @@ typedef enum stereo_format { void write_webm_file_header(struct WebmOutputContext *webm_ctx, const aom_codec_enc_cfg_t *cfg, - const struct aom_rational *fps, stereo_format_t stereo_fmt, unsigned int fourcc, const struct AvxRational *par); -- 2.50.0