mostly clang-tidy cleanups

[neheb]

No description provided.

[codecov]

Codecov Report

Merging #2124 (a8c8918) into main (b17828b) will decrease coverage by 0.02%.
The diff coverage is 65.51%.

❗ Current head a8c8918 differs from pull request most recent head a25bbe1. Consider uploading reports for the commit a25bbe1 to get more accurate results

@@            Coverage Diff             @@
##             main    #2124      +/-   ##
==========================================
- Coverage   63.41%   63.39%   -0.03%     
==========================================
  Files          97       97              
  Lines       19187    19190       +3     
  Branches     9721     9713       -8     
==========================================
- Hits        12168    12165       -3     
- Misses       4752     4756       +4     
- Partials     2267     2269       +2     

Impacted Files	Coverage Δ
include/exiv2/basicio.hpp	100.00% <ø> (ø)
src/casiomn_int.cpp	7.14% <0.00%> (ø)
src/cr2header_int.cpp	69.44% <ø> (ø)
src/epsimage.cpp	1.83% <0.00%> (ø)
src/exif.cpp	65.80% <0.00%> (-0.22%)	⬇️
src/fujimn_int.cpp	100.00% <ø> (ø)
src/http.cpp	0.00% <0.00%> (ø)
src/minoltamn_int.cpp	54.34% <ø> (ø)
src/nikonmn_int.cpp	58.73% <0.00%> (ø)
src/orfimage_int.cpp	33.33% <ø> (ø)
... and 125 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b17828b...a25bbe1. Read the comment docs.

[piponazo]

Thanks for the contribution! I just made some minor comments here and there, but most of the changes look good to me 😉

[neheb]

That should be it.

[piponazo]

LGTM, Thanks!