The cleanup step for removing redundant returns at the end of void
functions (added in commit 49c0bf4a) was flawed. Redundant returns have
a space (ie. 'return ;'), so adjust the regex to match that.

upstream fixed this variable name in 0.3.16.15 and this backports the
fix to make diffs between cleanup branches smaller.

Remove redundant TN40_FTRACE, FTRACE, FTRACE_ON and FTRACE_OFF macros.

(cherry picked from commit 68c5bf29)

void functions do not need to return; at the end. The empty return was
left over after removing the RET() macro in
f7344c42 and
3e916da0

(cherry picked from commit d9e29d13)

Lindent needs to be run at the start of the cleanup process, so that the
format is known and consistent, before the regex transformations are applied.

(cherry picked from commit 66ed7f24)

The test for remaining DBG1 macros (added in
 d7b9e9ab) incorrectly exited with
non-zero status and caused the cleanup.sh script to abort.

(cherry picked from commit 76d9bb10)

Remove comments like
 /* foo(bar); */
and
 /* foo->bar = baz; */
and
 /* foo = bar->baz; */

This doesn't get rid of all dead code. As usual the result was visually
checked for false positives to ensure that no accidental content
removals occurred.

(cherry picked from commit f2d0a35a)

(cherry picked from commit 3e916da0)

(cherry picked from commit 19e2c765)

(cherry picked from commit 2c06691d)

(cherry picked from commit 4380d741)

- move it to the cleanup directory
- let it build the bundled ccmtcnvt utility
- organise related steps into sections