[PATCH] Add support of callbacks for operation cancelation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[Please don't top-post, thanks.]

On Thu, 2013-01-03 at 15:27 -0500, Favonia wrote:
> Thanks. I'm still interested because we're working on a full Haskell
> binding for PulseAudio (in a very, very slow pace, though). I assume
> you're talking about an interface similar to
> pa_stream_set_state_callback(). If so, it looks more consistent with
> the current PulseAudio API. In either way we can deal with it much
> more efficiently than the old API.

Ok, would you be willing to write an updated patch?

Some feedback about the first version:

 * It would be better to call the callback from operation_set_state
rather than from context_unlink().

 * pa_operation_new_with_cancel_callback() seems unnecessary. There's no
need to initialize the callback to anything else than NULL when creating
a new operation object.

 * Every new function in the public API need to be added to
src/map-file, otherwise applications won't be able to use the function
(linking will fail).

-- 
Tanu



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux