On 30/04/2025 22:22, James Almer wrote:
> The output will be a fragment with zero units, which is a lot more user 
> friendly
> than making them think something went wrong, as it already happens with 
> cbs_av1.
> 
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
>  libavcodec/cbs_apv.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/cbs_apv.c b/libavcodec/cbs_apv.c
> index 2c4a1daaf0..e7dd04825c 100644
> --- a/libavcodec/cbs_apv.c
> +++ b/libavcodec/cbs_apv.c
> @@ -168,8 +168,8 @@ static int cbs_apv_split_fragment(CodedBitstreamContext 
> *ctx,
>      uint32_t signature;
>      int err, trace;
>  
> -    if (header) {
> -        // Ignore extradata fragments.
> +    if (header || !frag->data_size) {
> +        // Ignore empty or extradata fragments.
>          return 0;
>      }
>  

LGTM.

Thanks,

- Mark

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

Reply via email to