elFarto
commented
3 weeks ago I apologise for taking so long to get around to looking it this. Unfortunately that } you removed was required. The matching opening { is actually in the ARRAY_FOR_EACH macro, which probably isn't the best idea. That removal causes this patch not to compile. Given that, and the unfortunate reformatting, I won't be able to accept this PR. I will tidy up that mismatched } though.
The 'destory' typo has been fixed by another PR I've just merge.
When building the project as a Debian package from source, the Lintian compliance check was noisy at the "spellintian" phase related to this file. Ameliorated here are:
Document formatting applied by Artistic Style ('astyle') using the Google Open Source Code standards with minimal deviations.¹
① I apologize profusely for this, it should not have happened. I was in a hurry and normally would have disable my IDE's "format on save" directive but it didn't occur to me until milliseconds after I'd closed the editor and approved the save, leaving me with no undo history to fallback on. I completely understand if you cherry pick the changes you see fit and close this PR. My only dog in this fight is for Lintian to shut up about misspellings compiled into the binaries, as documented above.