[libcxx-commits] [PATCH] D60027: Add shift functions (P0769)

Zoe Carver via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sun Apr 7 17:45:19 PDT 2019


zoecarver marked an inline comment as done.
zoecarver added inline comments.


================
Comment at: include/algorithm:5806
+
+    for (unsigned __i = 0; __i < __n; ++__i, ++__first);
+
----------------
EricWF wrote:
> Can you write explicit braces to make it clear it's an empty loop?
For sure. A lot of the empty loops are formatted like this:
```
for (...)
    ;
```
Is that or braces preferred? 


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

https://reviews.llvm.org/D60027





More information about the libcxx-commits mailing list