[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