Eric Niebler comments

Results 206 comments of


                                            Eric Niebler

Fail with std::back_inserter

Range-v3 hasn't kept up with the changes made in `std::ranges`. This is mostly because I lack time. Patches welcome.

ranges::v3::view::interleave in docs but not in source code

It's tricky to make it perfectly general. The one in the calendar example makes some assumptions. I was trying to address the shortcomings when I got distracted, and that was...

iter_move for views that cycle

This seems like a bug. You shouldn't be able to move out of a cyclic range.

Major Performance Issues with view::concat

`concat` is much harder for a compiler to optimize. It iterates one range at a time, so the `concat` iterator only references one subrange's iterator at a time. In the...

range-v3 and boost

What version of Boost are you using? Range-v3 forward declares some of Boost.Ranges's templates. It's possible Boost.Range has changed recently.

the modulemap file is installed on /usr/include/module.modulemap causing downstream packaging problems

The modulemap was a contribution from @gnzlbg, and I know very little about the mechanics of clang's modules and modulemap files. @gnzlbg, can you comment please?

Ubuntu 20.10 g++10 -std=c++20 1-line crash

Please file this bug with gcc if you haven't already. A compiler crash is always a compiler bug. It's possible a workout exists. I'm not likely to have time to...

sample() of views::indices fails to compile with libstdc++ (because diffmax_t isn't an integral type)

I don't see how I can work around this in range-v3. You might be right, this could be a libstdc++ issue, or else an unfortunate but unavoidable(?) incompatibility.

std::span and views::span use different index type

A PR would be very appreciated, thanks.

Add support for Linux

@neo5167 The typical GitHub work flow is to create a fork of the repo, make your branch there, and make a pull request from there back to the original repo.