Skip to content

Commit

Permalink
Rollup merge of #64993 - mathstuf:backtrace-status-eq, r=withoutboats
Browse files Browse the repository at this point in the history
BacktraceStatus: add Eq impl

See discussion on #53487.

---
Is adding `Copy` too ambitious? It's a "status", so I don't forsee any non-POD data that might go in there, but it would restrict future variants more than `Eq` does.

Cc: @withoutboats @abonander
  • Loading branch information
Centril committed Oct 3, 2019
2 parents c4f8fd2 + fb80e6c commit f188879
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/libstd/backtrace.rs
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ pub struct Backtrace {
/// The current status of a backtrace, indicating whether it was captured or
/// whether it is empty for some other reason.
#[non_exhaustive]
#[derive(Debug)]
#[derive(Debug, PartialEq, Eq)]
pub enum BacktraceStatus {
/// Capturing a backtrace is not supported, likely because it's not
/// implemented for the current platform.
Expand Down

1 comment on commit f188879

@Lonami
Copy link
Contributor

@Lonami Lonami commented on f188879 Oct 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congratulations on the 100,000 commits!

Please sign in to comment.