[FFmpeg-devel] [PATCH] ffplay: rename audio_decode_frame() variable "pts" to "audio_clock0"

Stefano Sabatini stefasab at gmail.com
Sun Feb 3 17:11:57 CET 2013


On date Sunday 2013-02-03 16:42:44 +0100, Marton Balint encoded:
> On Sun, 3 Feb 2013, Alexander Strasser wrote:
[...]
> >> #ifdef DEBUG
> >>             {
> >>                 static double last_clock;
> >>-                printf("audio: delay=%0.3f clock=%0.3f pts=%0.3f\n",
> >>+                printf("audio: delay=%0.3f clock=%0.3f clock0=%0.3f\n",
> >>                        is->audio_clock - last_clock,
> >>-                       is->audio_clock, pts);
> >>+                       is->audio_clock, clock0);
> >                                          ^^^^^^
> >
> > Am I missing something or are you missing audio_ prefix over there?
> 
> Yes, that seems like a typo for me too :)
> 
> >
> >>                 last_clock = is->audio_clock;
> >
> > I cannot say if the new name is better but it sounds reasonable.
> 
> Other than the above, LGTM, thanks.

Sorry for that, fixed and applied.
-- 
FFmpeg = Fast and Fantastic Marvellous Programmable Elfic Goblin


More information about the ffmpeg-devel mailing list