[MPlayer-dev-eng] [PATCH] Re: [PATCH] output console status only every n frames (-quietframes), Bug #1459

[D. Jansen]

On Fri, May 29, 2009 at 11:25 PM, Uoti Urpala <uoti.urpala at pp1.inet.fi>wrote:

> I think it should be based on real time, not frames.
>
(...)

> Your suggestion for setting a default value based on fps wouldn't work
>  either, because a video does not necessarily have a fixed frame rate.
>

Thanks for the comment.

Is there agreement on that? Is there then be a chance that it gets
merged?

I think there wouldn't be much need for making it time-based,
as it's not necessary to get the output exactly x times a second,
but rather to simply decrease the amount of output.
And making the status output completely time based
and not frame based would mean a rewrite.

What I think I could do is to make e.g. a second
_highest_ output frequency. That would in deed enable us
to set a more sane default.

For now I'll attach the version that includes support for mencoder.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: quietframesv4+mencoder.patch
Type: text/x-diff
Size: 3157 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090531/18e98cf3/attachment.patch>