[vlc-devel] [PATCH] Added new commands in share/http/status.xml

[akash mehrotra]

On 29 March 2011 13:51, Mirsal Ennaime <mirsal at mirsal.fr> wrote:

> Hello,
>
> On Mon, 2011-03-28 at 23:01 +0300, Rémi Denis-Courmont wrote:
> > Le samedi 26 mars 2011 19:52:13 akash mehrotra, vous avez écrit :
> > > Added pl-forceplay, pl_forcepause, pl_resume to oldhttp interface
> > >
> > > Updated the relevant readme file
> > > pl_forcepause
> > >  pause playback, do nothing if paused
> > >
> > > pl_resume
> > >  resume playback, do nothing if playing
>
> As far as I have seen, commands like these ones have proven to be very
> useful to remote controllers in the case of the MPRIS.
>
> > How about a pl_setpause that accepts a boolean value? I think there's
> already
> > something along those lines in MPRIS.
>
> Actually, we have Pause() which is equivalent to pl_forcepause, Play()
> (roughly like pl_resume) and PlayPause() (pauses when playing, resumes
> when paused)
>
> see: http://www.mpris.org/2.1/spec/Player_Node.html
>
> (sorry if this is a bit off-topic, I do not wish to hijack this thread)
>
> --
> mirsal
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
> In the VLC http interface, pl_pause is already used for toggling(like
playpause),and pl_play will restart from beginning.(hence the need for using
new command names)

Which method would be better to implement these commands? Using pl_setpause
with a boolean argument, or 2 explicit commands for pause and resume?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110329/cf73b9ea/attachment.html>