On Thu May 30 14:41:45 2024 +0000, Matteo Bruni wrote:
More general style comments:
- we usually put the "release" calls in reverse order of allocation
- `if ()` body on its own line
BTW, definitely moving all the cleanup at the end of the function makes for nicer code, I approve :thumbsup:
Yes, the old way was pretty messy. The style issues should be fixed in the current revision.