Remove unused includes

[seladb]

No description provided.

[Dimi1010]

Just FYI, CI is failing because that all the headers like <cstring> only guarantee to declare their contents within the std namespace, while the old c headers like <string.h> only guarantee to have their contents in the global namespace. So some functions might need adjustments to use the std:: version.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.10%. Comparing base (44972d0) to head (76a7d54).
Report is 1 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #1604      +/-   ##
==========================================
- Coverage   83.12%   83.10%   -0.02%     
==========================================
  Files         276      276              
  Lines       46907    46907              
  Branches     9402     9536     +134     
==========================================
- Hits        38991    38983       -8     
- Misses       7060     7091      +31     
+ Partials      856      833      -23     

Flag	Coverage Δ
fedora40	74.67% <ø> (-0.05%)	⬇️
macos-12	81.09% <ø> (-0.01%)	⬇️
macos-13	80.53% <ø> (ø)
macos-14	80.45% <ø> (ø)
mingw32	70.63% <ø> (ø)
mingw64	70.60% <ø> (ø)
npcap	85.02% <ø> (ø)
rhel94	74.56% <ø> (-0.04%)	⬇️
ubuntu2004	57.93% <ø> (ø)
ubuntu2004-zstd	58.06% <ø> (+<0.01%)	⬆️
ubuntu2204	74.49% <ø> (+<0.01%)	⬆️
ubuntu2204-icpx	58.70% <ø> (ø)
ubuntu2404	74.74% <ø> (ø)
unittest	83.10% <ø> (-0.02%)	⬇️
windows-2019	85.05% <ø> (ø)
windows-2022	85.06% <ø> (ø)
winpcap	85.02% <ø> (-0.02%)	⬇️
xdp	49.66% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[seladb]

Just FYI, CI is failing because that all the headers like <cstring> only guarantee to declare their contents within the std namespace, while the old c headers like <string.h> only guarantee to have their contents in the global namespace. So some functions might need adjustments to use the std:: version.

Yes, I'm still working on this PR 🙂

[egecetin]

@seladb What about also adding https://github.com/include-what-you-use/include-what-you-use after fix all of them. I don't have experience with it, but do you already know or use it?

[tigercosmos]

LGTM

[seladb]

@seladb What about also adding https://github.com/include-what-you-use/include-what-you-use after fix all of them. I don't have experience with it, but do you already know or use it?

I've never used it, but I can look into it. Probably in a separate PR?

[egecetin]

@seladb Separate PR is better, I think. Because it might need additional config which will mix this PR.