Motivation / Problem

Similar to #8916, but with more.

This should give a clean build log for release builds too .. I hope :D

Description

Some parts could be rewritten to be nicer, this for sure was the lazy approach. But some code is just a bit horrible in terms of assert usage. Too close to a release to really worry about this.

Tested with both GCC 9 and CLang 10, both Debug and Release, all with asserts disabled.

It should make no functional changes .. but .. please look at this carefully :D

Limitations

Checklist for review

Some things are not automated, and forgotten often. This list is a reminder for the reviewers.

• The bug fix is important enough to be backported? (label: 'backport requested')
• This PR affects the save game format? (label 'savegame upgrade')
• This PR affects the GS/AI API? (label 'needs review: Script API')
  • ai_changelog.hpp, gs_changelog.hpp need updating.
  • The compatibility wrappers (compat_*.nut) need updating.
• This PR affects the NewGRF API? (label 'needs review: NewGRF')
  • newgrf_debug_data.h may need updating.
  • PR must be added to API tracker