On Mon Sep 18 17:52:43 2023 +0000, Zebediah Figura wrote:
The column limit is larger than 50 characters. No need to reflow to make it shorter. I'd advocate for keeping a trace in any API entry point. The 'result' variable is useless.
Okay, I thought it was easier to read the way I grouped the arguments based on their intent/use.
`normalize_old_file_image` is also called from the ApplyPatch* routines so having the trace in `normalize_old_file_image` really produces the most effective trace output.
I left the result variable for debugging purposes - no longer needed so I will remove it.