Sanctifier/clippy_errors.log at main · HyperSafeD/Sanctifier · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
warning: profiles for the non root package will be ignored, specify profiles at the workspace root:
package:   /home/rampop/Sanctifier/contracts/amm-pool/Cargo.toml
workspace: /home/rampop/Sanctifier/Cargo.toml
    Checking sanctifier-core v0.1.0 (/home/rampop/Sanctifier/tooling/sanctifier-core)
error: you should consider adding a `Default` implementation for `ArithmeticOverflowRule`
  --> tooling/sanctifier-core/src/rules/arithmetic_overflow.rs:10:5
   |
10 | /     pub fn new() -> Self {
11 | |         Self
12 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#new_without_default
   = note: `-D clippy::new-without-default` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(clippy::new_without_default)]`
help: try adding this
   |
 9 + impl Default for ArithmeticOverflowRule {
10 +     fn default() -> Self {
11 +         Self::new()
12 +     }
13 + }
   |

error: you should consider adding a `Default` implementation for `AuthGapRule`
  --> tooling/sanctifier-core/src/rules/auth_gap.rs:8:5
   |
 8 | /     pub fn new() -> Self {
 9 | |         Self
10 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#new_without_default
help: try adding this
   |
 7 + impl Default for AuthGapRule {
 8 +     fn default() -> Self {
 9 +         Self::new()
10 +     }
11 + }
   |

error: you should consider adding a `Default` implementation for `LedgerSizeRule`
  --> tooling/sanctifier-core/src/rules/ledger_size.rs:11:5
   |
11 | /     pub fn new() -> Self {
12 | |         Self {
13 | |             ledger_limit: 64000,
14 | |             approaching_threshold: 0.8,
...  |
17 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#new_without_default
help: try adding this
   |
10 + impl Default for LedgerSizeRule {
11 +     fn default() -> Self {
12 +         Self::new()
13 +     }
14 + }
   |

error: this `if` has identical blocks
   --> tooling/sanctifier-core/src/rules/ledger_size.rs:115:38
    |
115 |   ...   if size >= self.ledger_limit {
    |  ____________________________________^
116 | | ...       Some(SizeWarningLevel::Exceed...
117 | | ...   } else if self.strict_mode && siz...
    | |_______^
    |
note: same as this
   --> tooling/sanctifier-core/src/rules/ledger_size.rs:117:64
    |
117 |           } else if self.strict_mode && size >= strict_threshold {
    |  ________________________________________________________________^
118 | |             Some(SizeWarningLevel::ExceedsLimit)
119 | |         } else if size as f64 >= self.ledger_limit as f64 * self.ap...
    | |_________^
    = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#if_same_then_else
    = note: `-D clippy::if-same-then-else` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::if_same_then_else)]`

error: you should consider adding a `Default` implementation for `PanicDetectionRule`
 --> tooling/sanctifier-core/src/rules/panic_detection.rs:7:5
  |
7 | /     pub fn new() -> Self {
8 | |         Self
9 | |     }
  | |_____^
  |
  = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#new_without_default
help: try adding this
  |
6 + impl Default for PanicDetectionRule {
7 +     fn default() -> Self {
8 +         Self::new()
9 +     }
10+ }
  |

error: you should consider adding a `Default` implementation for `UnhandledResultRule`
  --> tooling/sanctifier-core/src/rules/unhandled_result.rs:9:5
   |
 9 | /     pub fn new() -> Self {
10 | |         Self
11 | |     }
   | |_____^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#new_without_default
help: try adding this
   |
 8 + impl Default for UnhandledResultRule {
 9 +     fn default() -> Self {
10 +         Self::new()
11 +     }
12 + }
   |

error: this `impl` can be derived
   --> tooling/sanctifier-core/src/lib.rs:196:1
    |
196 | / impl Default for RuleSeverity {
197 | |     fn default() -> Self {
198 | |         Self::Warning
199 | |     }
200 | | }
    | |_^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.93.0/index.html#derivable_impls
    = note: `-D clippy::derivable-impls` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::derivable_impls)]`
help: replace the manual implementation with a derive attribute and mark the default variant
    |
190 + #[derive(Default)]
191 | pub enum RuleSeverity {
192 |     Info,
193 ~     #[default]
194 ~     Warning,
    |

error: could not compile `sanctifier-core` (lib) due to 7 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `sanctifier-core` (lib test) due to 7 previous errors