sardemff7 / eventd

A simple daemon to track remote or local events and do actions the user wants to
https://www.eventd.org
Other
46 stars 4 forks source link

reconnect: add GDestroyNotify callback #7

Closed mathstuf closed 9 years ago

mathstuf commented 9 years ago

The user_data object can be allocated any way by the caller. Use GDestroyNotify to free the data.

Signed-off-by: Ben Boeckel mathstuf@gmail.com

sardemff7 commented 9 years ago

Please consider libeventd as internal API. If it is installed, it is only for a potential semi-official plugins repository that will be kept in sync.

mathstuf commented 9 years ago

Um. How so? The reconnect object logically owns the data, but has no way of knowing the proper way to free it. I don't think that has anything to do with private/public APIs.

sardemff7 commented 9 years ago

Commented inline for relay.c. And you forgot im.c which is also using the reconnect API.

mathstuf commented 9 years ago

Updated if it matters :) .

sardemff7 commented 9 years ago

libeventd is still internal API so no need for GDestroyNotify. ;-)