[vlc-devel] commit: Asynchronous timer API ( R??mi Denis-Courmont )

[Andre-John Mas]

On 3-Jun-2009, at 13:02, Rémi Denis-Courmont wrote:

>
>
>
> On Wed, 3 Jun 2009 12:45:10 +0200, jpd at videolan.org wrote:
>> On Tue, Jun 02, 2009 at 08:32:09PM +0200, git version control wrote:
>>> +struct vlc_timer_t
>>> +{
>>> +    timer_t handle;
>>> +    void (*func) (vlc_timer_t *, void *);
>>> +    void *data;
>>> +};
>>
>> Why func(vlc_timer_t, data) instead of just func(vlc_timer_t) and
>> expecting func to say vlc_timer_t->data?
>
> Because this is ugly. I could live with removing the timer, but  
> removing
> the opaque data seems wrong.

Looking in as an outsider, the first question that I would have is
what are the use cases, since this is what would define what the
best solutions are, and whether there is another approach to the
problem?

André-John