search

can-lehmann / owlkettle

A declarative user interface framework based on GTK 4
https://can-lehmann.github.io/owlkettle/README
MIT License
367 stars 14 forks source link

#76 add missing destructors #84

Closed PhilippMDoerner closed 9 months ago

PhilippMDoerner commented 9 months ago

Adds destructors to the cstring-returning procs from gtk defined in the table in one of my comments in #76

This is a first attempt

can-lehmann commented 9 months ago

I think that we can merge this after removing the echo calls. Also the indentation of the comment is still weird.

can-lehmann commented 9 months ago

Looks good :+1:

PhilippMDoerner commented 9 months ago

Removed the echo's and the comment indentation. That one I originally had in there to not disrupt your organization via comments that used them to divide your bindings into sections based on the file the bound symbol came from.

Note that I still didn't test that the =sink hook gets called anywhere because I couldn't come up with the code for that. Did manage a copy though and that one does get called as expected.