Version 0.6.1

Fixes #9

Cargo.lock

@@ -295,7 +295,7 @@ dependencies = [
 
 [[package]]
 name = "decomp-toolkit"
-version = "0.6.0"
+version = "0.6.1"
 dependencies = [
  "anyhow",
  "ar",

Cargo.toml

@@ -3,7 +3,7 @@ name = "decomp-toolkit"
 description = "Yet another GameCube/Wii decompilation toolkit."
 authors = ["Luke Street <luke@street.dev>"]
 license = "MIT OR Apache-2.0"
-version = "0.6.0"
+version = "0.6.1"
 edition = "2021"
 publish = false
 repository = "https://github.com/encounter/decomp-toolkit"

src/util/dwarf.rs

@@ -373,20 +373,19 @@ impl Tag {
     pub fn children<'a>(&self, tags: &'a TagMap) -> Vec<&'a Tag> {
         let sibling = self.next_sibling(tags);
         let mut children = Vec::new();
-        let (_, mut child) = match tags.range(self.key + 1..).next() {
+        let mut child = match self.next_tag(tags) {
             Some(child) => child,
             None => return children,
         };
-        if child.kind == TagKind::Padding {
-            return children;
-        }
         loop {
             if let Some(end) = sibling {
                 if child.key == end.key {
                     break;
                 }
             }
-            children.push(child);
+            if child.kind != TagKind::Padding {
+                children.push(child);
+            }
             match child.next_sibling(tags) {
                 Some(next) => child = next,
                 None => break,
@@ -395,15 +394,18 @@ impl Tag {
         children
     }
 
+    /// Returns the next sibling tag, if any
     pub fn next_sibling<'a>(&self, tags: &'a TagMap) -> Option<&'a Tag> {
         if let Some(key) = self.reference_attribute(AttributeKind::Sibling) {
-            if let Some(next) = tags.get(&key) {
-                if next.kind != TagKind::Padding {
-                    return Some(next);
-                }
-            }
+            tags.get(&key)
+        } else {
+            self.next_tag(tags)
         }
-        None
+    }
+
+    /// Returns the next tag sequentially, if any
+    pub fn next_tag<'a>(&self, tags: &'a TagMap) -> Option<&'a Tag> {
+        tags.range(self.key + 1..).next().map(|(_, tag)| tag)
     }
 }