Re: [FFmpeg-devel] [PATCH 1/2] lavc/vvc_parser: small cleanup for style

2024-03-23 Thread Nuo Mi
On Fri, Mar 22, 2024 at 6:36 PM Jun Zhao  wrote:

> From: Jun Zhao 
>
> small cleanup for style, redundant semicolons, goto labels,
> in FFmpeg, we put goto labels at brace level.
>
Pushed this patch.
Thanks, Jun


> Signed-off-by: Jun Zhao 
> ---
>  libavcodec/vvc_parser.c | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
> index c661595e1e..245cb214c9 100644
> --- a/libavcodec/vvc_parser.c
> +++ b/libavcodec/vvc_parser.c
> @@ -173,7 +173,7 @@ static void set_parser_ctx(AVCodecParserContext *s,
> AVCodecContext *avctx,
>  h266_sub_width_c[sps->sps_chroma_format_idc];
>  s->height = pps->pps_pic_height_in_luma_samples -
>  (pps->pps_conf_win_top_offset + pps->pps_conf_win_bottom_offset) *
> -h266_sub_height_c[sps->sps_chroma_format_idc];;
> +h266_sub_height_c[sps->sps_chroma_format_idc];
>
>  avctx->profile = sps->profile_tier_level.general_profile_idc;
>  avctx->level = sps->profile_tier_level.general_level_idc;
> @@ -317,7 +317,7 @@ static int get_pu_info(PuInfo *info, const
> CodedBitstreamH266Context *h266,
>  }
>  info->pic_type = get_pict_type(pu);
>  return 0;
> -  error:
> +error:
>  memset(info, 0, sizeof(*info));
>  return ret;
>  }
> @@ -329,7 +329,7 @@ static int append_au(AVPacket *pkt, const uint8_t
> *buf, int buf_size)
>  if ((ret = av_grow_packet(pkt, buf_size)) < 0)
>  goto end;
>  memcpy(pkt->data + offset, buf, buf_size);
> -  end:
> +end:
>  return ret;
>  }
>
> @@ -376,7 +376,7 @@ static int parse_nal_units(AVCodecParserContext *s,
> const uint8_t *buf,
>  } else {
>  ret = 1; //not a completed au
>  }
> -  end:
> +end:
>  ff_cbs_fragment_reset(pu);
>  return ret;
>  }
> --
> 2.25.1


> ___
> 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".
>
___
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".


[FFmpeg-devel] [PATCH 1/2] lavc/vvc_parser: small cleanup for style

2024-03-22 Thread Jun Zhao
From: Jun Zhao 

small cleanup for style, redundant semicolons, goto labels,
in FFmpeg, we put goto labels at brace level.

Signed-off-by: Jun Zhao 
---
 libavcodec/vvc_parser.c | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index c661595e1e..245cb214c9 100644
--- a/libavcodec/vvc_parser.c
+++ b/libavcodec/vvc_parser.c
@@ -173,7 +173,7 @@ static void set_parser_ctx(AVCodecParserContext *s, 
AVCodecContext *avctx,
 h266_sub_width_c[sps->sps_chroma_format_idc];
 s->height = pps->pps_pic_height_in_luma_samples -
 (pps->pps_conf_win_top_offset + pps->pps_conf_win_bottom_offset) *
-h266_sub_height_c[sps->sps_chroma_format_idc];;
+h266_sub_height_c[sps->sps_chroma_format_idc];
 
 avctx->profile = sps->profile_tier_level.general_profile_idc;
 avctx->level = sps->profile_tier_level.general_level_idc;
@@ -317,7 +317,7 @@ static int get_pu_info(PuInfo *info, const 
CodedBitstreamH266Context *h266,
 }
 info->pic_type = get_pict_type(pu);
 return 0;
-  error:
+error:
 memset(info, 0, sizeof(*info));
 return ret;
 }
@@ -329,7 +329,7 @@ static int append_au(AVPacket *pkt, const uint8_t *buf, int 
buf_size)
 if ((ret = av_grow_packet(pkt, buf_size)) < 0)
 goto end;
 memcpy(pkt->data + offset, buf, buf_size);
-  end:
+end:
 return ret;
 }
 
@@ -376,7 +376,7 @@ static int parse_nal_units(AVCodecParserContext *s, const 
uint8_t *buf,
 } else {
 ret = 1; //not a completed au
 }
-  end:
+end:
 ff_cbs_fragment_reset(pu);
 return ret;
 }
-- 
2.25.1

___
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".