src: use ranges library (C++20) to simplify code

[lemire]

This PR uses C++20 ranges to slightly simplify a piece of code. To be honest, the real motivation of this PR is to introduce ranges and test whether the platforms play nice with ranges. If so, then this could be generalized in the code base.

[github-actions]

Failed to start CI

   ⚠  Commits were pushed since the last approving review:
   ⚠  - src: use ranges library (C++20) to simplify code
   ⚠  - breaking long lines
   ✘  Refusing to run CI on potentially unsafe PR

https://github.com/nodejs/node/actions/runs/14598958420

[richardlau]

Looks like a missing #include <algorithm>:
https://github.com/nodejs/node/actions/runs/14598914061/job/40951831889?pr=57975#step:6:6555

In file included from ../src/node_options.cc:2:
../src/node_options-inl.h:398:20: error: no member named 'for_each' in namespace 'std::ranges'
  398 |       std::ranges::for_each(
      |       ~~~~~~~~~~~~~^

[lemire]

@richardlau good catch.

[anonrig]

@lemire It seems it doesn't build on any platforms at all

[lemire]

@anonrig

It seems it doesn't build on any platforms at all

It should build. Let us find out what the issue is.

[codecov]

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.27%. Comparing base (367d593) to head (8289bba).
Report is 17 commits behind head on main.

Files with missing lines	Patch %	Lines
src/node_options-inl.h	88.88%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #57975      +/-   ##
==========================================
- Coverage   90.28%   90.27%   -0.02%     
==========================================
  Files         630      630              
  Lines      186169   186169              
  Branches    36479    36479              
==========================================
- Hits       168082   168062      -20     
- Misses      10971    10974       +3     
- Partials     7116     7133      +17     

Files with missing lines	Coverage Δ
src/node_options-inl.h	81.30% <88.88%> (+0.48%)	⬆️

... and 33 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[lemire]

@anonrig I am unhappy with the formatting of the code, but it works now !!!!!!!!!

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/66433/

[nodejs-github-bot]

Landed in 25fe802