[MPlayer-dev-eng] [PATCH] Patch for better pause support.

Robert Cummings robert at interjinn.com
Thu Sep 4 05:19:10 CEST 2008


On Tue, 2008-09-02 at 20:49 +0200, Reimar Döffinger wrote:
>
> Please check if attached patch does everything you need. I intend to
> apply in two days and after someone confirms that they find it useful this
> way.

Attached patch has a couple of deficiencies, only one of which is
important. Specifically it doesn't set the pause status before entry
into run_command(). The other issue is that the new keyword doesn't
handle unpausing as my original patch did. This is a moot point though
since the behaviour can easily be had by not using pausing_keep_force.
The attached patch adds 1 line to yours to provide the necessary pause
status information.

I noticed also you're patch doesn't contain my addition of the is_paused
command. I've attached this as a second patch so they can be applied
independently. The is_paused command allows probing for the current
pause status.

Cheers,
Rob.
-- 
http://www.interjinn.com
Application and Templating Framework for PHP
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pkf.new.diff
Type: text/x-patch
Size: 1778 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080903/4ca13fbb/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: is_paused.patch
Type: text/x-patch
Size: 1435 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080903/4ca13fbb/attachment-0001.bin>


More information about the MPlayer-dev-eng mailing list