From: Behdad Esfahbod <behdad@behdad.org>
Date: Tue, 7 May 2019 18:05:51 +0000 (-0700)
Subject: [serialize] Use range-based loop
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=530ddbbc320bd24b4902ee6d49bf80242a591794;p=platform%2Fupstream%2FlibHarfBuzzSharp.git

[serialize] Use range-based loop
---

diff --git a/src/hb-serialize.hh b/src/hb-serialize.hh
index afaa4c4..1a6ed21 100644
--- a/src/hb-serialize.hh
+++ b/src/hb-serialize.hh
@@ -287,26 +287,23 @@ struct hb_serialize_context_t
   {
     assert (!current);
 
-    for (auto obj_it = ++hb_iter (packed); obj_it; ++obj_it)
+    for (const object_t *parent : ++hb_iter (packed))
     {
-      const object_t &parent = **obj_it;
-
-      for (auto link_it = parent.links.iter (); link_it; ++link_it)
+      for (const object_t::link_t &link : parent->links)
       {
-        const object_t::link_t &link = *link_it;
 	const object_t &child = *packed[link.objidx];
-	//assert (link.bias <= parent.tail - parent.head);
-	unsigned offset = (child.head - parent.head) - link.bias;
+	//assert (link.bias <= parent->tail - parent->head);
+	unsigned offset = (child.head - parent->head) - link.bias;
 
 	if (link.is_wide)
 	{
-	  auto &off = * ((BEInt<uint32_t, 4> *) (parent.head + link.position));
+	  auto &off = * ((BEInt<uint32_t, 4> *) (parent->head + link.position));
 	  assert (0 == off);
 	  check_assign (off, offset);
 	}
 	else
 	{
-	  auto &off = * ((BEInt<uint16_t, 2> *) (parent.head + link.position));
+	  auto &off = * ((BEInt<uint16_t, 2> *) (parent->head + link.position));
 	  assert (0 == off);
 	  check_assign (off, offset);
 	}