Updating code for suggestions from clang-tidy - #120

Merged

andrewdavidsmith merged 13 commits into

mainfrom

clang-tidy-updates

Aug 22, 2026

Commits on Aug 21, 2026

src/falco_utils.hcpp: some simplifcation of type names and using span to avoid unchecked array bounds issues; additional simplifications
andrewdavidsmith
committed
b42db9b
src/html.cpp: replacing direct array access with spans in some functions
andrewdavidsmith
committed
7c40645
src/results_collector.hpp and src/results_summary.hpp: replacing gc_content_array with gc_content_t because it's not an array anymore
andrewdavidsmith
committed
6efb530
src/kmer_counter.cpp: removing some nolints and replacing the 'bases' var in decode_kmer with a global one that's used elsewhere though in fewer places than exptected
andrewdavidsmith
committed
2682e9e
src/duplication_results.cpp: more changes suggested by static analysis
andrewdavidsmith
committed
5b1f36d
src/report.cpp: changes suggested by clang-tidy
andrewdavidsmith
committed
c568cf2
src/html.cpp: reverting the previous change related to colors for plotting sequence content
andrewdavidsmith
committed
65bb7ac
src/falco_config.cpp: some simplification in load_config_and_set_graders function
andrewdavidsmith
committed
3e8746d
src/falco_utils.hpp: defining static constexpr global structures as arrays but declaring spans to use them without direct array access
andrewdavidsmith
committed
b3c89f3