On Tue, Jun 07, 2011 at 10:13:01AM -0400, Justin Ruggles wrote:
> On 06/07/2011 04:48 AM, Diego Biurrun wrote:
>
> > On Mon, Jun 06, 2011 at 05:09:42PM +0200, Diego Biurrun wrote:
> >> [...]
> >
> > Before anybody asks...
> >
> >> --- a/ffplay.c
> >> +++ b/ffplay.c
> >> @@ -1065,10 +1065,6 @@ static double compute_target_time(double
> >> frame_current_pts, VideoState *is)
> >> is->frame_timer += delay;
> >> -#if defined(DEBUG_SYNC)
> >> - printf("video: delay=%0.3f actual_delay=%0.3f pts=%0.3f A-V=%f\n",
> >> - delay, actual_delay, frame_current_pts, -diff);
> >> -#endif
> >
> > CC ffplay.o
> > ffplay.c: In function ‘compute_target_time’:
> > ffplay.c:1070:20: error: ‘actual_delay’ undeclared (first use in this
> > function)
> > ffplay.c:1070:20: note: each undeclared identifier is reported only once
> > for each function it appears in
>
> might be useful. just remove actual_delay and change it to av_dlog().
Done.
> >> --- a/libavcodec/motion_est.c
> >> +++ b/libavcodec/motion_est.c
> >> @@ -1119,8 +1119,6 @@ void ff_estimate_p_frame_motion(MpegEncContext * s,
> >> - av_dlog(s, "varc=%4d avg_var=%4d (sum=%4d) vard=%4d mx=%2d my=%2d\n",
> >> - varc, s->avg_mb_var, sum, vard, mx - xx, my - yy);
> >
> > CC libavcodec/motion_est.o
> > libavcodec/motion_est.c: In function ‘ff_estimate_p_frame_motion’:
> > libavcodec/motion_est.c:1122:5: error: ‘MpegEncContext’ has no member named
> > ‘avg_mb_var’
> > libavcodec/motion_est.c:1122:5: error: ‘xx’ undeclared (first use in this
> > function)
> > libavcodec/motion_est.c:1122:5: note: each undeclared identifier is
> > reported only once for each function it appears in
> > libavcodec/motion_est.c:1122:5: error: ‘yy’ undeclared (first use in this
> > function)
> > make: *** [libavcodec/motion_est.o] Error 1
>
> This could be changed to:
> av_dlog(s, "varc=%4d sum=%4d vard=%4d mx=%2d my=%2d\n",
> varc, sum, vard, mx, my);
>
> but it hasn't compiled since 2002 so I won't be sad if it goes away.
>
> >> --- a/libavformat/mpeg.c
> >> +++ b/libavformat/mpeg.c
> >> @@ -583,9 +583,6 @@ static int64_t mpegps_read_dts(AVFormatContext *s, int
> >> stream_index,
> >>
> >> pos = *ppos;
> >> -#ifdef DEBUG_SEEK
> >> - printf("read_dts: pos=0x%"PRIx64" next=%d -> ", pos, find_next);
> >> -#endif
> >
> > CC libavformat/mpeg.o
> > libavformat/mpeg.c: In function ‘mpegps_read_dts’:
> > libavformat/mpeg.c:587:5: error: implicit declaration of function
> > ‘please_use_av_log_instead_of_printf’
> > [-Werror=implicit-function-declaration]
> > libavformat/mpeg.c:587:58: error: ‘find_next’ undeclared (first use in this
> > function)
> > libavformat/mpeg.c:587:58: note: each undeclared identifier is reported
> > only once for each function it appears in
>
> ok.
I removed these two.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
