diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2022-09-19 10:25:03 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-19 10:25:03 +0200 |
| commit | b1b418b81a13ede548273665e83c1bc5a97dffcd (patch) | |
| tree | 731efb5a2811e1d925e94e9a8a16c220ac98de92 /core | |
| parent | a4a894fa1e933e8a678f8ad4f6353837859b02fd (diff) | |
chore: fix clippy warnings (#15944)
Stop allowing clippy::derive-partial-eq-without-eq and fix warnings
about deriving PartialEq without also deriving Eq.
In one case I removed the PartialEq because it a) wasn't necessary,
and b) sketchy because it was comparing floating point numbers.
IMO, that's a good argument for enforcing the lint rule, because it
would most likely have been caught during review if it had been enabled.
Diffstat (limited to 'core')
| -rw-r--r-- | core/error.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/error.rs b/core/error.rs index 3a36a6ab9..7af6ad375 100644 --- a/core/error.rs +++ b/core/error.rs @@ -134,7 +134,7 @@ pub struct JsError { pub aggregated: Option<Vec<JsError>>, } -#[derive(Debug, PartialEq, Clone, serde::Deserialize, serde::Serialize)] +#[derive(Debug, Eq, PartialEq, Clone, serde::Deserialize, serde::Serialize)] #[serde(rename_all = "camelCase")] pub struct JsStackFrame { pub type_name: Option<String>, |