Re: [FFmpeg-devel] [PATCH v5 02/10] avcodec: add cbs for H266/VVC

From: Thomas Siedel <thomas.ff@spin-digital.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v5 02/10] avcodec: add cbs for H266/VVC
Date: Fri, 10 Feb 2023 19:06:21 +0100
Message-ID: <CAD25kL42q8NCLCi9rfc8DY0SC4QYtaKLRxWfZamASueof2uVmA@mail.gmail.com> (raw)
In-Reply-To: <tencent_5E20574DC84FD3D50BA772B89A16024D3A07@qq.com>

On Wed, 1 Feb 2023 at 09:21, "zhilizhao(赵志立)" <quinkblack@foxmail.com>
wrote:

>
>
> > On Jan 3, 2023, at 21:40, Thomas Siedel <thomas.ff@spin-digital.com>
> wrote:
> ...
> > diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c
> > index 4ee06003c3..8ae8ae6cda 100644
> > --- a/libavcodec/cbs_h2645.c
> > +++ b/libavcodec/cbs_h2645.c
> > @@ -24,10 +24,16 @@
> > #include "cbs_internal.h"
> > #include "cbs_h264.h"
> > #include "cbs_h265.h"
> > +#include "cbs_h266.h"
> > #include "h264.h"
> > #include "h2645_parse.h"
> > #include "hevc.h"
> > +#include "vvc.h"
> >
> > +static av_always_inline unsigned int h266_ceil(unsigned int v, unsigned
> int align)
> > +{
> > +    return (((v) + (align) - 1) / (align));
> > +}
>
> Does AV_CEIL_RSHIFT() work for the usecase?
>
>
Yes, it can be used instead. Thank you for the suggestion.
I changed it now in the new version of the patch set.
_______________________________________________
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".