Skip to content

[arcane,core] Use a sentinel instead of iterator for ItemLocalIdListViewT::end() - #2823

Merged
grospelliergilles merged 1 commit into
mainfrom
dev/gg-use-sentinel-for-end-of-itemlocalidlistview-iterator
Oct 14, 2026
Merged

grospelliergilles merged 1 commit into
mainfrom
dev/gg-use-sentinel-for-end-of-itemlocalidlistview-iterator

Conversation

@grospelliergilles

Copy link
Copy Markdown
Member

A sentinel uses less memory and only have a comparison operator.
It is only available since C++20.

…ViewT::end()'.

A sentinel uses less memory and only have a comparison operator.
It is only available since C++20.
@grospelliergilles grospelliergilles self-assigned this Oct 14, 2026
@grospelliergilles grospelliergilles added arcane Arcane Component refactoring Code refactoring and cleanup labels Oct 14, 2026
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.41%. Comparing base (dcb961c) to head (af3d8b1).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2823      +/-   ##
==========================================
- Coverage   71.44%   71.41%   -0.04%     
==========================================
  Files        2574     2574              
  Lines      194789   194781       -8     
  Branches    23093    23090       -3     
==========================================
- Hits       139176   139105      -71     
- Misses      46971    47030      +59     
- Partials     8642     8646       +4     

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@grospelliergilles
grospelliergilles merged commit 32242bd into main Oct 14, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arcane Arcane Component refactoring Code refactoring and cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant