[FFmpeg-devel] [PATCH] ffplay show status and refresh
Michael Niedermayer
michaelni
Thu Jul 30 09:06:15 CEST 2009
On Wed, Jul 29, 2009 at 10:02:44PM -0700, Baptiste Coudurier wrote:
> Hi guys,
>
> Always show status and increase refresh time "a la" mplayer, it looks nice
> this way :>
>
> Don't remove the option, it could break users using it :)
>
> --
> Baptiste COUDURIER GnuPG Key Id: 0x5C1ABAAA
> Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
> FFmpeg maintainer http://www.ffmpeg.org
> ffplay.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
> b932c51e74b18f91f418336478d23d1e9fc62a4d ffplay_status.patch
> Index: ffplay.c
> ===================================================================
> --- ffplay.c (revision 19507)
> +++ ffplay.c (working copy)
> @@ -195,7 +195,7 @@
> static int wanted_subtitle_stream= -1;
> static int seek_by_bytes;
> static int display_disable;
> -static int show_status;
> +static int show_status = 1;
> static int av_sync_type = AV_SYNC_AUDIO_MASTER;
> static int64_t start_time = AV_NOPTS_VALUE;
> static int debug = 0;
> @@ -1138,7 +1138,7 @@
> double av_diff;
>
> cur_time = av_gettime();
> - if (!last_time || (cur_time - last_time) >= 500 * 1000) {
> + if (!last_time || (cur_time - last_time) >= 1000) {
i think displaying this 1000 times per second is too much though 2
times is too little
except that iam ok with the patch
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Old school: Use the lowest level language in which you can solve the problem
conveniently.
New school: Use the highest level language in which the latest supercomputer
can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090730/00128889/attachment.pgp>
More information about the ffmpeg-devel
mailing list