On Mon Sep 18 18:15:44 2023 +0000, Jeffrey Smith wrote:
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;`
I agree it is unnecessary. I wrote it as such for quick emulation of the Windows behavior. I will rewrite this to make it more clear.