On Wed, May 6, 2020 at 5:40 PM Martin Storsjö <mar...@martin.st> wrote: > > On Wed, 6 May 2020, Linjie Fu wrote: > > > Allows specifying avctx->profile with string type profile for > > h264 encoders. > > > > Private field/option "profile" may be abled to be removed for basic > > h264 profiles, directly for encoders like libopenh264/h264_vaapi, > > or with an map to hardware profile like h264_qsv. > > > > Signed-off-by: Linjie Fu <linjie...@intel.com> > > --- > > One concern is some encoders have options for specific profiles, > > like "high444p" for nvenc_h264 and "constrained_high" for h264_amf, > > hence they may not be able to remove the private option easily. > > > > Please help to comment. > > > > libavcodec/options_table.h | 4 ++++ > > 1 file changed, 4 insertions(+) > > This change in itself looks sensible to me (but I'd like for someone else > to comment as well). Even if it might not be able to get rid of the > private option for all encoders, it should at least simplify the easiest > cases. > > // Martin I think we have a discussion about this case, https://patchwork.ffmpeg.org/project/ffmpeg/patch/fa14da65-9e1a-c74b-b3cb-46fc5f3ea...@gmail.com/ _______________________________________________ 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".
