[FFmpeg-devel] [PATCH] ffplay: rename temporary variable pts in audio_decode_frame()
Stefano Sabatini
stefasab at gmail.com
Sun Feb 3 17:09:33 CET 2013
On date Sunday 2013-02-03 16:40:23 +0100, Marton Balint encoded:
>
>
> On Sun, 3 Feb 2013, Stefano Sabatini wrote:
>
> >On date Saturday 2013-02-02 22:20:41 +0100, Marton Balint encoded:
> >>
> >>
> >>On Sat, 2 Feb 2013, Stefano Sabatini wrote:
> >>
> >>>On date Saturday 2013-02-02 16:17:11 +0100, Stefano Sabatini encoded:
> >>>>Also rename variable pts to pts_time.
> >>>
> >>>s/rename/remove/ in the subject.
> >>>
> >>>>---
> >>>>ffplay.c | 16 +++++++---------
> >>>>1 file changed, 7 insertions(+), 9 deletions(-)
> >>
> >>Unless I am missing something, the pts_time (former pts_ptr) output
> >>of audio_decode_frame is not used anywhere, so why not remove the
> >>whole pts_ptr, pts or pts_time variables from both
> >>sdl_audio_callback and audio_decode_frame?
> >
> >Up.
>
> LGTM, thanks,
Applied, thanks.
--
FFmpeg = Faithful Formidable Majestic Perfectionist Extravagant Gorilla
More information about the ffmpeg-devel
mailing list