[vlc-devel] [PATCH 2/2] libvlc: add event libvlc_MediaPlayerESChanged

Rémi Denis-Courmont remi at remlab.net
Wed Jul 16 17:12:30 CEST 2014


Le 2014-07-16 17:41, Edward Wang a écrit :
> As for the issue at hand: if only the ID is provided, then it does 
> not
> indicate whether the ID was added or removed.

Of course it does. The ID will be in one of the ES list if it was added 
or udpated, and will not if it was deleted. Obviously it would be much 
better to have three separate event types respectively for add, change 
and remove.

> Since that information is not provided, we have to pull the new
> versions of all 3 lists to compare and find the ID in question to 
> find
> out.

At least you can do it.

> By providing the ES type instead, we would only need to pull the one
> relevant list instead of all 3 lists.

And you can do nothing with the list since you do know neither if you 
are looking for a new, existing or gone entry, and for which entry 
anyway.

Of course, it's better if the ES type is supplied in the event, but 
that datum is less informative/critical than the ES ID.

-- 
Rémi Denis-Courmont



More information about the vlc-devel mailing list