On Mon Sep 18 17:52:49 2023 +0000, Zebediah Figura wrote:
Why are we adding one here? That doesn't look right.
I think I see what the intent is, but it is unnecessarily 'clever'. It would be clearer to use something like a `BOOL modified`, and ultimately `return modified ? NORMALIZE_RESULT_SUCCESS_MODIFIED : NORMALIZE_RESULT_SUCCESS;`