[GEN][ZH] Fix and simplify trailing CR LF in most logging related functions and macros

[xezon]

Merge with Rebase

• Fixes DEBUG_LOG, DEBUG_CRASH & Co need their caller strings to end on "\n" #1021

This change fixes and simplifies the trailing Carriage Return \r and Line Feed \n in most logging related functions and macros. The trailing \n is now appended in the logging wrapper itself, and so user code no longer needs to add it. This makes it less error prone.

The only disadvantage is that it gives no finer control over calling DEBUG_LOG multiple times to write to the same log line. However, that is a rare use case. If such a log is needed in the future, we can add a new DEBUG_LOG_RAW macro or so.

Affected macros are:

• DEBUG_LOG
• DEBUG_LOG_LEVEL
• DEBUG_CRASH
• DEBUG_ASSERTLOG
• DEBUG_ASSERTCRASH
• RELEASE_CRASH
• RELEASE_CRASHLOCALIZED
• WWDEBUG_SAY
• WWDEBUG_WARNING
• WWRELEASE_SAY
• WWRELEASE_WARNING
• WWRELEASE_ERROR
• WWASSERT_PRINT
• WWDEBUG_ERROR
• SNAPSHOT_SAY
• SHATTER_DEBUG_SAY
• DBGMSG
• REALLY_VERBOSE_LOG
• DOUBLE_DEBUG
• PERF_LOG
• CRCGEN_LOG
• STATECHANGED_LOG
• PING_LOG
• BONEPOS_LOG

Some logging in WWVegas ended with \r\n (CR LF) which has been fixed as well.

Most log lines have been fixed with a script. Some lines have been fixed and polished by hand. I have labeled the commits accordingly.

[roossienb]

I'll review this, although I'm not yet sure about the best approach consider its size

[xezon]

I mainly reviewed it by checking left-over \n and \r

[xezon]

Rebased with Main. Fixed some mixups in 2 commits.

[aliendroid1]

you can try generating the preprocessed files and diffing them. Assuming everything here is done correctly, they should be identical

[roossienb]

Some log entries contained double newline characters, which led to either a single lingering newline or completely blank log messages. Both cases are formatting artifacts and should be cleaned up to maintain log file consistency.

Hopefully I’ve tracked down all instances—fatigue was definitely setting in toward the end.

[roossienb]

Remove lineend

[xezon]

I think EA placed this intentionally to make the log block stand out more.

[xezon]

Reverted.

[xezon]

Rebased on Main.

I addressed all comments and some more.

I appended changes with fixup commits, which will be squashed later.

[xezon]

I have tested that individual commits build.