[CodeHealth] Fix clang-tidy warnings part 2

[dbarker]

Contributes to #2053

Changes

Fixes warnings in the following categories:

• bugprone-signed-char-misuse
• bugprone-suspicious-missing-comma
• bugprone-forward-declaration-namespace
• bugprone-use-after-move
• cppcoreguidelines-explicit-virtual-functions
• cppcoreguidelines-rvalue-reference-param-not-moved
• cppcoreguidelines-use-default-member-init
• misc-unused-parameters
• performance-move-const-arg
• performance-unnecessary-value-param
• performance-avoid-endl

For significant contributions please make sure you have completed the following items:

• CHANGELOG.md updated for non-trivial changes
• Unit tests have been added
• Changes in public API reviewed

[netlify]

✅ Deploy Preview for opentelemetry-cpp-api-docs canceled.

Name	Link
🔨 Latest commit	22ca025
🔍 Latest deploy log	https://app.netlify.com/projects/opentelemetry-cpp-api-docs/deploys/685cf03a8d82d90008eb9b48

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.96%. Comparing base (8465778) to head (22ca025).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3496      +/-   ##
==========================================
- Coverage   89.99%   89.96%   -0.02%     
==========================================
  Files         219      219              
  Lines        7039     7049      +10     
==========================================
+ Hits         6334     6341       +7     
- Misses        705      708       +3     

Files with missing lines	Coverage Δ
exporters/ostream/test/ostream_log_test.cc	96.45% <100.00%> (+0.28%)	⬆️
sdk/src/common/global_log_handler.cc	76.20% <100.00%> (-1.08%)	⬇️
sdk/src/trace/span.cc	89.78% <ø> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[marcalff]

LGTM, thanks for the cleanup.