commit 606841b07017ac80dea2fc5ada25b5976f2f9192
author Behdad Esfahbod <behdad@behdad.org> Sat May 11 11:54:30 2019 -0700
committer Behdad Esfahbod <behdad@behdad.org> Sat May 11 11:56:26 2019 -0700
tree 54f66195cb89f83cf9919b94e73fc93021874ca4
parent c1c122e7b3f60dc7b5a224c68d2acb106fda8b49

[iter] Check for more before forwarding/rewinding past ends

src/hb-iter.hh

diff --git a/src/hb-iter.hh b/src/hb-iter.hh
index 1b5c427..dcc49ee 100644
--- a/src/hb-iter.hh
+++ b/src/hb-iter.hh
@@ -196,11 +196,11 @@
 
   /* Advancing: Implement __next__(), or __forward__() if random-access. */
   void __next__ () { *thiz() += 1; }
-  void __forward__ (unsigned n) { while (n--) ++*thiz(); }
+  void __forward__ (unsigned n) { while (*thiz() && n--) ++*thiz(); }
 
   /* Rewinding: Implement __prev__() or __rewind__() if bidirectional. */
   void __prev__ () { *thiz() -= 1; }
-  void __rewind__ (unsigned n) { while (n--) --*thiz(); }
+  void __rewind__ (unsigned n) { while (*thiz() && n--) --*thiz(); }
 
   /* Range-based for: Implement __end__() if can be done faster,
    * and operator!=. */

src/test-iter.cc

diff --git a/src/test-iter.cc b/src/test-iter.cc
index d9e2a97..f834640 100644
--- a/src/test-iter.cc
+++ b/src/test-iter.cc
@@ -82,10 +82,8 @@
     (void) _;
 
   it += it.len ();
-  if (0)
-    it = it + 10;
-  if (0)
-    it = 10 + it;
+  it = it + 10;
+  it = 10 + it;
 
   assert (*it == it[0]);