From fecec6293f622c1df36dbfd131d34956b4b712d8 Mon Sep 17 00:00:00 2001 From: James Zern Date: Mon, 15 Aug 2022 17:26:41 -0700 Subject: [PATCH] simple_encode.cc: clear -Wextra-semi-stmt warnings fixes warnings of the form: ../vp9/simple_encode.cc:755:48: warning: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Wextra-semi-stmt] SET_STRUCT_VALUE(config, oxcf, ret, key_freq); Bug: b/229626362 Change-Id: I1c9b0ae9927cdd7c31da000633bcb6e2b8242cd4 --- vp9/simple_encode.cc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/vp9/simple_encode.cc b/vp9/simple_encode.cc index 654699e1b..f42912d35 100644 --- a/vp9/simple_encode.cc +++ b/vp9/simple_encode.cc @@ -744,10 +744,12 @@ static void UpdateGroupOfPicture(const VP9_COMP *cpi, int start_coding_index, } #define SET_STRUCT_VALUE(config, structure, ret, field) \ - if (strcmp(config.name, #field) == 0) { \ - structure->field = atoi(config.value); \ - ret = 1; \ - } + do { \ + if (strcmp(config.name, #field) == 0) { \ + structure->field = atoi(config.value); \ + ret = 1; \ + } \ + } while (false) static void UpdateEncodeConfig(const EncodeConfig &config, VP9EncoderConfig *oxcf) { -- 2.50.0