[FFmpeg-devel] [PATCH] h264: put call to ff_print_debug_info2 under CONFIG_MPEGVIDEO.

Michael Niedermayer michaelni at gmx.at
Sun Mar 3 04:06:35 CET 2013


On Sat, Mar 02, 2013 at 05:03:38PM -0800, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje" <rsbultje at gmail.com>
> 
> The code is located in mpegvideo, and it's likely that in a minimal
> config, we don't want to include debug info anyway.
> ---
>  libavcodec/h264.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/h264.c b/libavcodec/h264.c
> index 9d86e01..659c023 100644
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -4842,8 +4842,10 @@ not_extra:
>  
>      assert(pict->data[0] || !*got_frame);
>  
> -    ff_print_debug_info2(h->avctx, pict, h->er.mbskip_table, h->visualization_buffer, &h->low_delay,
> -                         h->mb_width, h->mb_height, h->mb_stride, 1);

> +    if (CONFIG_MPEGVIDEO) {

I think CONFIG_MPEGVIDEO is missing in mplayer too

patch LGTM

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

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130303/1400f494/attachment.asc>


More information about the ffmpeg-devel mailing list