James Almer: > On 6/29/2023 3:17 PM, Andreas Rheinhardt wrote: >> Fixes a warning from GCC when in pedantic mode: >> "extra semicolon in struct or union specified" >> >> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com> >> --- >> libavcodec/cbs_h266.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/cbs_h266.h b/libavcodec/cbs_h266.h >> index 8548e501c0..03dfd4a954 100644 >> --- a/libavcodec/cbs_h266.h >> +++ b/libavcodec/cbs_h266.h >> @@ -80,7 +80,7 @@ typedef struct H266GeneralConstraintsInfo { >> /* inter */ >> uint8_t gci_no_ref_pic_resampling_constraint_flag; >> - uint8_t gci_no_res_change_in_clvs_constraint_flag;; >> + uint8_t gci_no_res_change_in_clvs_constraint_flag; >> uint8_t gci_no_weighted_prediction_constraint_flag; >> uint8_t gci_no_ref_wraparound_constraint_flag; >> uint8_t gci_no_temporal_mvp_constraint_flag; >> @@ -211,7 +211,7 @@ typedef struct H266RawVUI { >> uint8_t vui_aspect_ratio_idc; >> uint16_t vui_sar_width; >> - uint16_t vui_sar_height;; >> + uint16_t vui_sar_height; >> uint8_t vui_overscan_info_present_flag; >> uint8_t vui_overscan_appropriate_flag; > > LGTM. This should also fix a compilation failure with msvc, which seems > to be pretty strict about this. > http://fate.ffmpeg.org/report.cgi?time=20230629174305&slot=x86_64-msvc17-windows-native
I'll amend the commit message and apply the patchset. - Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".