Re: [FFmpeg-devel] [PATCH 3/3] avcodec/utils: print Chroma Location string in verbose log level

2016-10-16 Thread James Almer 
On 10/16/2016 11:32 AM, Michael Niedermayer wrote:
> On Sat, Oct 15, 2016 at 12:40:56PM -0300, James Almer wrote:
>> It's container level information on some formats (Matroska, MXF, yuv4mpeg), 
>> so
>> it should be printed at higher log levels than debug.
>>
>> Signed-off-by: James Almer 
>> ---
>>  libavcodec/utils.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> LGTM
> 
> thx

Pushed, thanks.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/utils: print Chroma Location string in verbose log level

2016-10-16 Thread Michael Niedermayer 
On Sat, Oct 15, 2016 at 12:40:56PM -0300, James Almer wrote:
> It's container level information on some formats (Matroska, MXF, yuv4mpeg), so
> it should be printed at higher log levels than debug.
> 
> Signed-off-by: James Almer 
> ---
>  libavcodec/utils.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

LGTM

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel