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

[akash mehrotra]

2011/3/22 Rémi Denis-Courmont <remi at remlab.net>

> Le lundi 21 mars 2011 20:22:40 akash mehrotra, vous avez écrit :
> > > This is a bit confused description. The implementation seems to be
> > > race-prone.
> > >
> > > Didn't really understand how its race prone. The only difference from
> the
> >
> > original play code is that a check has been added, and avoids resetting
> the
> > video if called while the video is paused.
>
> Exactly, that seems like a time-of-check/time-of-use race to me.
>
>
> --
> Rémi Denis-Courmont
> Looking for a job: http://www.remlab.info/
> http://fi.linkedin.com/in/remidenis
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>


The code is quite similar to the pl_pause code, which already exists.
I would like to change the behaviour of pl_play, but if it has the
possibility of causing problems, then I will remove that change and send in
a new patch.
However,I would appreciate it if you could explain what scenario would lead
to a race condition?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110322/b73c5bdf/attachment.html>