Skip to content

Commit af19d68

Browse files
romtsnclaude
andcommitted
ref(proguard): Mark unreachable Proguard branch in get_problem()
Since is_usable() now always returns true for Proguard mappings, the Proguard arm in get_problem() is dead code. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 7a41802 commit af19d68

File tree

1 file changed

+8
-14
lines changed

1 file changed

+8
-14
lines changed

src/utils/dif.rs

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -380,24 +380,18 @@ impl<'a> DifFile<'a> {
380380
pub fn is_usable(&self) -> bool {
381381
match self {
382382
DifFile::Archive(_) => self.has_ids() && self.features().has_some(),
383-
DifFile::Proguard(..) => true,
383+
_ => true,
384384
}
385385
}
386386

387387
pub fn get_problem(&self) -> Option<&'static str> {
388-
if self.is_usable() {
389-
None
390-
} else {
391-
Some(match self {
392-
DifFile::Archive(..) => {
393-
if !self.has_ids() {
394-
"missing debug identifier, likely stripped"
395-
} else {
396-
"missing debug or unwind information"
397-
}
398-
}
399-
DifFile::Proguard(..) => "missing line information",
400-
})
388+
match self {
389+
DifFile::Archive(..) if !self.is_usable() => Some(if !self.has_ids() {
390+
"missing debug identifier, likely stripped"
391+
} else {
392+
"missing debug or unwind information"
393+
}),
394+
_ => None,
401395
}
402396
}
403397

0 commit comments

Comments
 (0)