Eliminate unnecessary argument from FunctionTracer and derived classes

[abeltrano]

Type

• Bug fix
• Feature addition
• Feature update
• Documentation
• Build Infrastructure

Side Effects

• Breaking change
• Non-functional change

Goals

• Reduce the number of arguments needed when customizing function tracing.

Technical Details

• The logSeverityExit argument in the FunctionTracer constructor didn't make a lot of sense since it would be unusual for callers to want to use a different exit log severity from the enter log severity when the object is first created. Thus, the logSeverityExit argument is removed (including from all derived classes), and the logSeverityStart argument is renamed to logSeverity, and its value used for both function enter and exit traces.

Test Results

• Ran unit tests and ensured the same log verbosity prior to this change was maintained.

Reviewer Focus

• Consider whether the assumption made above

Future Work

• None

Checklist

• Build target all compiles cleanly.
• clang-format and clang-tidy deltas produced no new output.
• Newly added functions include doxygen-style comment block.