[FFmpeg-devel] FFplay: progress bar feature proposal

[Raymond Pierce]

13.04.2017, 09:40, "Steven Liu" <lingjiujianke at gmail.com>:
> 2017-04-13 20:48 GMT+08:00 Raymond Pierce <ray1110000 at yandex.com>:
>
>>  Hi. Currently FFplay has no visible progress bar and it is hard to tell
>>  where a stream is currently positioned. So if one wants, for example, to
>>  rewind
>>  a little back relative to current position using right mouse click it is
>>  always
>>  guessing and trying.
>>
>>  I propose very simple 1-pixel progress bar which can be turned on or off
>>  during
>>  playback by pressing a button (I use 'L', from 'line'). By default the bar
>>  is off.
>>
>>  I choose bright green color (#00ff00) for the part to the left of a current
>>  position and pure black for the part to the right.
>>
>>  You can see how it looks on the attached image.
>>
>>  Draft patch is applied. One global variable ('static int
>>  show_progress_line')
>>  and one function ('static void video_progress_line_display(VideoState
>>  *is)')
>>  have been added. Also the existing 'video_display()' function has been
>>  moved
>>  below 'get_master_clock()' as the latter is used in the new function.
>>
>>  What do you think?
>>  _______________________________________________
>>  ffmpeg-devel mailing list
>>  ffmpeg-devel at ffmpeg.org
>>  http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> git commit -a
> git format-patch -s -1
>
> You should use the command looks like the above git command to make patch
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Hello, Steven. Thank you for the advice. New patch in the attachment.
Should I post a new message with the patch to ffmpeg-devel at ffmpeg.org list?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-FFplay-progress-bar-feature.patch
Type: text/x-diff
Size: 4152 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170413/7f70756e/attachment.patch>