[vlc-devel] Re: Find an error in vlm.c

[ye zhang]

Sorry for my late reply!

I have tested the patch and the problem can not be reprodued any more.
But I doubt there is still some problem in the patch.
There is a p_media->i_instance-- in fuction vlm_MediaControl.
So if you use p_media->i_instance-- to replace i--,  p_media->i_instance--
will be done twice.

Thanks advance!
Ye Zhang

On 9/28/06, Jean-Paul Saman <jean-paul.saman at planet.nl> wrote:
>
> Jean-Paul Saman wrote:
> > Jean-Baptiste KEMPF wrote:
>
> >
> > I doubt the solution as presented above is correct. Examining the code I
> > wonder if it shouln't be p_media->i_instance-- iso j--;
> >
> > Could you run a test with p_media->i_instance--; ?
> On second thought my solution above is not complete it should be like in
> the attached patch. Could you please rerun the test with the attach
> patch applied? And let me know if the problem can be reproduced with the
> patch or not.
>
> Gtz,
> Jean-Paul Saman.
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20060928/8ccb3c2e/attachment.html>