[vlc-devel] [PATCH] ProjectM API (libvlc)
xilasz at gmail.com
Sun Dec 26 18:48:48 CET 2010
> It's not completely obvious to me what the lock is meant for. I think you
> should explain it in Doxygen.
ok, i'll add some details in doxygen (projectm change the preset every 30s,
lock is meant to prevent the change and keep the current preset).
I think different variable names for each command would be clearer for VLC
> developers, even if this is not visible to external LibVLC applications.
ok, i'll do that, and remove #define too.
> I gave the p_filter to the callback at its creation.
> > So libvlc set/get variables to the aout, then the aout callback set/get
> > variables to the filter, and the filter react and do his job.
> > If projectm is not used, variables doesn't exists in the aout, so nothing
> > happens.
> There seems to be a thread-safety problem here. The callback cannot access
> projectM pointer directly. Similarly, b_changed and i_preset_id lack memory
That's what i was afraid of (even if it works flawlessly), unfortunatelly
i'm kind of new to thread-safety problems in vlc (i know the possible
issues, i don't know how to fix them), so i would need help to fix this.
Can you help me ?
> I also added 3 settings to projectm module to set meshx, meshy and
> > texturesize, and fixed a crash if there is no available presets.
> That's probably good (I don't know projectM much), but it should be a
i'll send a separate patch for this soon.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel