[FFmpeg-devel] [PATCH 4/6] avutil/bprint: use AV_BPRINT_SIZE_AUTOMATIC instead of 1

lance.lmwang at gmail.com lance.lmwang at gmail.com
Fri Jun 26 13:39:57 EEST 2020


On Fri, Jun 26, 2020 at 07:17:01AM +0200, Nicolas George wrote:
> lance.lmwang at gmail.com (12020-06-26):
> > From: Limin Wang <lance.lmwang at gmail.com>
> > 
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > ---
> >  libavutil/bprint.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> No to gratuitous changes that pollute the log for no benefit:
> AV_BPRINT_SIZE_AUTOMATIC = 1 is documented and part of the API.

Sorry, I'm not clear why not to use 1 directly as it's public API?


> 
> Regards,
> 
> -- 
>   Nicolas George



-- 
Thanks,
Limin Wang


More information about the ffmpeg-devel mailing list