On Fri, Apr 10, 2015 at 04:55:55PM -0400, Derek Buitenhuis wrote: > From: James Almer <[email protected]> > > Based on code from libavcodec/libx264.c > > Signed-off-by: James Almer <[email protected]> > Signed-off-by: Derek Buitenhuis <[email protected]> > --- > libavcodec/libx265.c | 15 ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-)
Again, just "libx265" is enough as a msg prefix. > --- a/libavcodec/libx265.c > +++ b/libavcodec/libx265.c > @@ -100,7 +100,20 @@ static av_cold int libx265_encode_init(AVCodecContext > *avctx) > + av_log(avctx, AV_LOG_ERROR, "Error setting preset/tune %s/%s.\n", > ctx->preset, ctx->tune); > + av_log(avctx, AV_LOG_INFO, "Possible presets:"); > + av_log(avctx, AV_LOG_INFO, "Possible tunes:"); "Supported presets/tunes" would be clearer IMO. For my ears there exist things that are possible (somewhere) and supported (here). LGTM otherwise. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
