Clean up the code
Some things I noticed:
- We don't use
g_auto/g_autofree/g_autoptr
- Static functions have
g_return_(val_)if_fail()
- Callbacks are named whatever
- vfunc overrides don't have namespace_class_ prefix.
-
gpointer userdata
in signal callbacks + manual cast later instead of an implicit cast via parameter type
And probably a lot more.