[libcxx-commits] [PATCH] D122074: [libc++] Allow an output_iterator in fill.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante planned changes to this revision.
Mordante added a comment.

Thanks for the feedback!
In that case I put this patch on hold. In `format` I'm not only using `fill`, but several other algorithms like `copy` and `transform`. So only doing one algorithm makes little sense. I only did this one patch to float the idea. Especially `copy` is a more effort due to the wrap iterator.

As mentioned on Discord when I started to work on this patch these algorithms in ranges were still far off, now they are not that far off. When I restarted working on it recently I didn't really think about the ranges versions anymore until @philnik asked an excellent question.

I like your suggestion about writing a paper. I was surprised these iterators in `<algorithm>` didn't change in C++20. But I can't recall seeing a paper suggesting this direction. I'll have a look whether there has been a paper, if not I will consider writing one.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122074/new/

https://reviews.llvm.org/D122074