Currently there is a pointer to the songinformation in the
VisActorPlugin itself. This is set within VisActor by
doing: actplugin->songinfo = songinfo....
I don't think this is nice, not even when a method
function is used instead of the pointer assignment.
I would rather do plugin->render (plugin, video, audio, songinfo);
Ofcourse most use is encapsulated using the more abstract VisActor
interface but I think this would be a good change.
Do you people agree ?
|
