commit 1002a3dcd3ac50894e8b4b82ae1daef6277be13d
author Garret Rieger <grieger@google.com> Thu Jul 28 20:17:36 2022 +0000
committer Garret Rieger <grieger@google.com> Fri Jul 29 22:21:31 2022 +0000
tree 7553faf44f54cb56c5faef86198d63896aab7918
parent a5c2c8c1319fcc4d776496ebed05ed6c69060a96

[repacker] bug fixes.

src/graph/graph.hh

diff --git a/src/graph/graph.hh b/src/graph/graph.hh
index 234a261..7dc5413 100644
--- a/src/graph/graph.hh
+++ b/src/graph/graph.hh
@@ -80,13 +80,19 @@
       }
     }
 
-    void remove_real_link (unsigned child_index)
+    void remove_real_link (unsigned child_index, const void* offset)
     {
       for (unsigned i = 0; i < obj.real_links.length; i++)
       {
-        if (obj.real_links[i].objidx != child_index) continue;
+        auto& link = obj.real_links[i];
+        if (link.objidx != child_index)
+          continue;
+
+        if ((obj.head + link.position) != offset)
+          continue;
+
         obj.real_links.remove (i);
-        break;
+        return;
       }
     }
 
@@ -544,15 +550,12 @@
     auto* new_link = new_v.obj.real_links.push ();
     new_link->width = O::static_size;
     new_link->objidx = child_id;
-    new_link->is_signed = 0;
-    new_link->whence = 0;
-    new_link->position = (const char*) old_offset - (const char*) new_v.obj.head;
-    new_link->bias = 0;
+    new_link->position = (const char*) new_offset - (const char*) new_v.obj.head;
 
     auto& child = vertices_[child_id];
     child.parents.push (new_parent_idx);
 
-    old_v.remove_real_link (child_id); // TODO: needs to include offset to ensure correct one removed
+    old_v.remove_real_link (child_id, old_offset); // TODO: needs to include offset to ensure correct one removed
     child.remove_parent (old_parent_idx);
   }