SocketListener::accept_async panics

[antoyo]

Hi.
The current SocketListener::accept_async method does not work because the Object parameter of the callback should be optional (and the .gir file says so) but it is not.
It might be because gir does not look at the optional attribute when generating the callback.

[EPashkin]

Gir don't look at this attribute at all, only at allow-none but don't trusts him too, as many cases when it wrong.
In current realization of async trampoline IMHO will be touchcomplicated adding standard config checking as trampoline not analyzed.

[sdroege]

For the async trampoline we should maybe reverse the rules and assume by default that everything can be NULL unless overridden.

The problem basically is that GIR annotations exist only for "this can be NULL", but it can be easily forgotten in C libraries so the absence of that attribute does not necessarily mean that it must not be NULL / can't be NULL. So for return values we always assume that they can be NULL unless overridden in the toml file, for arguments we assume they must not be NULL unless the attribute is set (or overridden in the toml file).
What would help is a "this can not be NULL" annotation for return values.

[sdroege]

For now maybe just implement this specific function manually?

[sdroege]

@antoyo What's the status here? Is there still a problem, and if so can you provide a testcase to reproduce it?

[sdroege]

It is. Fixing

[sdroege]

See #221