[FFmpeg-devel] [PATCH] ffplay: rename temporary variable pts in audio_decode_frame()

Marton Balint cus at passwd.hu
Sun Feb 3 16:40:23 CET 2013



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,

Marton


More information about the ffmpeg-devel mailing list