Initial binding cleanup using generics

[Jacalz]

Description:

I honestly should be studying but I couldn't hold my self back with this one. The cleanup gains are too substantial.

For #5378

Checklist:

• Tests included.
• Lint and formatter run with no errors.
• Tests all pass.

[coveralls]

coverage: 59.034% (-0.2%) from 59.189%
when pulling 04ae66f on Jacalz:binding-cleanup-generic
into d7eeddc on fyne-io:develop.

[andydotxyz]

Thanks for this, it looks good indeed - cool stuff.

I've made a comment inline as the test changes seem unnecessary.

On a side note (unimportant) the gen script could have been updated to generate all the API in its original place leading to a clearer diff. With this approach we have to rely on the tests to verify that the function signatures are the same as they were before.

[andydotxyz]

Isn't this considered bad practice? I thought I read you should only increment as each item is added

[andydotxyz]

Anyhow I'd be tempted to say that in a refactor like this I'd prefer to see no test changes ;)

[Jacalz]

Yeah, this seems like some sort of thing that I accidentally committed after some race fix testing. I have no idea what is the best practice. The official example adds one at a time but I also can't find anything to say that adding them all at once would be bad. It is certainly faster 😅

[Jacalz]

Thanks. Yeah, I noticed that too late and the test change was not meant to be there. Will fix :)

[andydotxyz]

Sweet thanks so much!