commit 69c6928289cad871b5b096fbdcd2827fdb80c3ad
author Behdad Esfahbod <behdad@behdad.org> Wed Jun 07 14:11:02 2023 -0600
committer Behdad Esfahbod <behdad@behdad.org> Wed Jun 07 14:11:02 2023 -0600
tree 2e6bdcc76baf9810c2120cf716286ad15ccd25ff
parent 138461beb474cc65fe4b967efbd12b4e7753ead1

[iter] Another try at writing some for loops as range loops

src/hb-iter.hh

diff --git a/src/hb-iter.hh b/src/hb-iter.hh
index 98d65a8..f50af39 100644
--- a/src/hb-iter.hh
+++ b/src/hb-iter.hh
@@ -496,8 +496,8 @@
   operator () (Iter it)
   {
     AccuT value = init_value;
-    for (; it; ++it)
-      value = r (value, *it);
+    for (auto&& _ : it)
+      value = r (value, _);
     return value;
   }
 
@@ -679,8 +679,8 @@
 	    hb_requires (hb_is_iterator (Iter))>
   void operator () (Iter it)
   {
-    for (; it; ++it)
-      (void) hb_invoke (a, *it);
+    for (auto&& _ : it)
+      (void) hb_invoke (a, _);
   }
 
   private:
@@ -879,8 +879,8 @@
 	    hb_requires (hb_is_iterator (Iter))>
   void operator () (Iter it)
   {
-    for (; it; ++it)
-      s << *it;
+    for (auto&& _ : it)
+      s << _;
   }
 
   private:
@@ -906,8 +906,8 @@
 	    hb_requires (hb_is_iterator (Iter))>
   void operator () (Iter it) const
   {
-    for (; it; ++it)
-      (void) *it;
+    for (auto&& _ : it)
+      (void) _;
   }
 }
 HB_FUNCOBJ (hb_drain);
@@ -923,11 +923,10 @@
 	    hb_requires (hb_is_iterator (Iter))>
   void operator () (Iter it)
   {
-    for (; it; ++it)
+    for (auto&& _ : it)
     {
-      const auto &v = *it;
-      s1 << v.first;
-      s2 << v.second;
+      s1 << _.first;
+      s2 << _.second;
     }
   }