<div dir="ltr">4. Not sure if it would be better to change <i>return iota(...) </i>to <i>return views::iota(...)</i>.</div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">Hewill Kang <<a href="mailto:hewillk@gmail.com">hewillk@gmail.com</a>> 於 2025年10月14日 週二 下午11:32寫道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">1. The prefix <i>ranges::</i> in <i>ranges::__detail::__is_integer_like</i> can be removed.<br>2. Parameters should not be passed by reference; otherwise, c<i>onst int i = 0; std::views::indices(i);</i> will fail because <i>const int&</i> does not satisfy <i>__is_integer_like</i>.<div>3. operator() can be static.</div><div><br><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Yuao Ma <<a href="mailto:addr2line@gmail.com" target="_blank">addr2line@gmail.com</a>> 於 2025年10月14日 週二 下午11:14寫道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Jonathan,<br>
<br>
On Tue, Oct 14, 2025 at 10:55 PM Jonathan Wakely <<a href="mailto:jwakely@redhat.com" target="_blank">jwakely@redhat.com</a>> wrote:<br>
> Attaching patches as application/octet-stream makes them hard to<br>
> review and comment inline. If possible using 'git send-email' is the<br>
> ideal way to submit patches. If that's not possible (it can be awkward<br>
> to set up to send via gmail) then attaching a .txt file as text/plain<br>
> makes things easier for reviewers.<br>
><br>
<br>
I'm attaching it as a txt file now; hopefully, this works.<br>
<br>
> Putting the [[nodiscard]] attribute after operator() was necessary in<br>
> previous releases of GCC due to -fconcepts-ts compatibility, but<br>
> that's no longer relevant for GCC 16. You can just put [[nodiscard]]<br>
> before the return type.<br>
><br>
<br>
Done.<br>
<br>
> If you're contributing under the DCO terms then please don't add the<br>
> FSF copyright notice to new tests. And the test is not very novel or<br>
> interesting, so the licence text isn't needed either, see<br>
> <a href="https://gcc.gnu.org/onlinedocs/libstdc++/manual/test.html#test.new_tests" rel="noreferrer" target="_blank">https://gcc.gnu.org/onlinedocs/libstdc++/manual/test.html#test.new_tests</a><br>
><br>
<br>
Thanks for the guidance. I've removed the copyright now.<br>
<br>
><br>
> Also, I think this can be unconditionally noexcept. We know that<br>
> constructing iota_view from an integer-like type will not throw.<br>
><br>
> For an arbitrary program-defined type that models weakly_incrementable<br>
> it could throw, but not for integer-like types.<br>
><br>
<br>
Done.<br>
<br>
Thanks for the prompt review!<br>
Yuao<br>
</blockquote></div>
</blockquote></div>