Skip to content

Commit 54fc812

Browse files
authored
Merge pull request #279 from dtolnay/contextbacktrace
Remove 2 frames of noise from 'context' backtraces
2 parents f2123ab + 131249b commit 54fc812

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

src/context.rs

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,15 +47,23 @@ where
4747
where
4848
C: Display + Send + Sync + 'static,
4949
{
50-
self.map_err(|error| error.ext_context(context))
50+
// Not using map_err to save 2 useless frames off the captured backtrace
51+
// in ext_context.
52+
match self {
53+
Ok(ok) => Ok(ok),
54+
Err(error) => Err(error.ext_context(context)),
55+
}
5156
}
5257

5358
fn with_context<C, F>(self, context: F) -> Result<T, Error>
5459
where
5560
C: Display + Send + Sync + 'static,
5661
F: FnOnce() -> C,
5762
{
58-
self.map_err(|error| error.ext_context(context()))
63+
match self {
64+
Ok(ok) => Ok(ok),
65+
Err(error) => Err(error.ext_context(context())),
66+
}
5967
}
6068
}
6169

0 commit comments

Comments
 (0)