[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