Re: [FFmpeg-devel] [PATCH 1/4] avcodec/cbs_h266: Remove double ;

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

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.rheinhardt@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-request@ffmpeg.org with subject "unsubscribe".