Skip to content
Snippets Groups Projects

[bugfix] Fix two bugs in TransformedRangeView and TransformedRangeIterator

Merged Carsten Gräser requested to merge bugfix/fix-transformed-range-iterator into master
All threads resolved!
  • Properly forward r-value references of functions to enable move-construction for function stored in TransformedRangeView where possible.
  • Fix and test operator-> in TransformedRangeIterator. For transformations returning l-value references, the operator did a copy, such that it->foo() and (*it).foo() act on different objects. The fix is to correctly forward the l-value reference to the proxy-object returned by operator->.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 2 commits

    • de38fa63 - [bugfix] Fix operator-> for non-proxy TransformedRangeIterator
    • ef56d403 - [test] Add check for TransformedRangeIterator::operator->

    Compare with previous version

  • Carsten Gräser resolved all threads

    resolved all threads

  • Carsten Gräser resolved all threads

    resolved all threads

  • Carsten Gräser enabled an automatic merge when the pipeline for ef56d403 succeeds

    enabled an automatic merge when the pipeline for ef56d403 succeeds

  • mentioned in commit b71d7bf3

  • Please register or sign in to reply
    Loading