On Thu, 19 Jan 2012 10:01:53 -0500, Justin Ruggles <[email protected]> 
wrote:
> On 01/19/2012 02:28 AM, Anton Khirnov wrote:
> 
> > 
> > On Wed, 18 Jan 2012 17:19:42 -0500, Justin Ruggles 
> > <[email protected]> wrote:
> >> ---
> >>  libavcodec/utils.c |    4 +++-
> >>  1 files changed, 3 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> >> index ff3f065..3fa9a55 100644
> >> --- a/libavcodec/utils.c
> >> +++ b/libavcodec/utils.c
> >> @@ -1405,7 +1405,9 @@ void avcodec_string(char *buf, int buf_size, 
> >> AVCodecContext *enc, int encode)
> >>      int bitrate;
> >>      AVRational display_aspect_ratio;
> >>  
> >> -    if (encode)
> >> +    if (enc->codec)
> >> +        p = enc->codec;
> >> +    else if (encode)
> >>          p = avcodec_find_encoder(enc->codec_id);
> >>      else
> >>          p = avcodec_find_decoder(enc->codec_id);
> >> -- 
> >> 1.7.1
> >>
> > 
> > Can you elaborate in the commit message about the purpose/effect of
> > this?
> 
> 
> Sure. How about:
> 
> avformat: Use AVCodecContext.codec for displaying the codec name
> 
> If there is more than one codec available for the codec_id,
> use the one already selected by the user rather than the
> default codec returned by avcodec_find_(de/en)coder.
> 

Yeah, looks good.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to