Address PR review feedback

- Reverse pending_frames and pop from end instead of remove(0)
- Add comment explaining intentional Some(0)/None asymmetry
- Use let-else with .first() instead of is_empty + indexing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: romtsn

src/cache/mod.rs

@@ -899,7 +899,7 @@ impl<'r, 'data> RemappedFrameIter<'r, 'data> {
     fn next_inner(&mut self) -> Option<StackFrame<'data>> {
         // Drain any buffered frames from multi-frame expansion first.
         if !self.pending_frames.is_empty() {
-            return Some(self.pending_frames.remove(0));
+            return self.pending_frames.pop();
         }
 
         if let Some(frame) = self.fallback.take() {
@@ -918,8 +918,8 @@ impl<'r, 'data> RemappedFrameIter<'r, 'data> {
                     members.as_slice(),
                     self.outer_source_file,
                 );
-                if !frames.is_empty() {
-                    let first = frames.remove(0);
+                frames.reverse();
+                if let Some(first) = frames.pop() {
                     self.pending_frames = frames;
                     return Some(first);
                 }
@@ -1134,8 +1134,8 @@ fn resolve_no_line_frames<'a>(
 /// - **0:0 entries** (`startline().is_some()`): if any entry has a range
 ///   (`original_endline != original_startline`), all emit `Some(0)`;
 ///   otherwise each emits its `original_startline`.
-/// - **No-range entries** (`startline().is_none()`): a single entry uses
-///   its `original_startline` (or `Some(0)` for bare methods); multiple entries
+/// - **No-range entries** (`startline().is_none()`): a single entry emits
+///   its `original_startline` if > 0, otherwise `Some(0)`; multiple entries
 ///   with the same name collapse to one frame with `Some(0)`; different names
 ///   each emit `Some(0)` in original order.
 fn resolve_base_entries<'a>(

src/mapper.rs

@@ -309,10 +309,9 @@ fn resolve_no_line_frames<'s>(
     }
 
     // No base entries — fall back to all entries with output line 0.
-    if mapping_entries.is_empty() {
+    let Some(first) = mapping_entries.first() else {
         return;
-    }
-    let first = &mapping_entries[0];
+    };
     let unambiguous = mapping_entries.iter().all(|m| m.original == first.original);
     if unambiguous {
         collected
@@ -335,8 +334,8 @@ fn resolve_no_line_frames<'s>(
 /// - **0:0 entries** (`startline.is_some()`): if any entry has a range
 ///   (`original_endline != original_startline`), all emit `Some(0)`;
 ///   otherwise each emits its `original_startline`.
-/// - **No-range entries** (`startline.is_none()`): a single entry uses
-///   its `original_startline` (or `Some(0)` for bare methods); multiple entries
+/// - **No-range entries** (`startline.is_none()`): a single entry emits
+///   its `original_startline` if > 0, otherwise `Some(0)`; multiple entries
 ///   with the same name collapse to one frame with `Some(0)`; different names
 ///   each emit `Some(0)` in original order.
 fn resolve_base_entries<'s>(
@@ -392,10 +391,8 @@ fn resolve_base_entries<'s>(
                     Some(0)
                 } else if member.original_startline.unwrap_or(0) > 0 {
                     member.original_startline
-                } else if member.original_startline.is_none() {
-                    Some(0)
                 } else {
-                    None
+                    Some(0)
                 };
                 collected
                     .frames