« first day (3033 days earlier)      last day (455 days later) » 

8:07 AM
Clippy makes me sad. The rust format for number literals very conveniently allows you to write numbers in a readable way, suited to the usage. And clippy breaks it by refusing 0 prefixed literals...
 
8:26 AM
@DenysSéguret I don't fully get it, you like to write 0_112_452 ?
 
9:16 AM
There are many cases. Here I've just made a small macro in my tests to initiate date_time like d!(2026, 02, 28). Padding months or days is more readable.
More generally, the liberty given by rust number literals is very great and it's sad to see Clippy try to regulate that. I don't care about the old programmer dumb enough to try to put a 0 for octal or I don't know what.
 
9:41 AM
haha true octat that start with 0 was a very bad idea
 
Yeah, we can just forget that instead of considering it a norm
 
 
5 hours later…
2:56 PM
regrettably there is still software in use that accepts 031 and turns it into 25
This happened two days ago to a coworker: they put the cursor in a text field that had an initial value of 0, added two 9s, and hit Enter
but it had no effect because 099 isn't a valid octal number
 
Yes, I know. But I mean it shouldn't be considered a norm enough to think a Rust programmer would type a 0 to create an octal literal. If we don't stop at some point we'll never get rid of this.
 
oh, I agree, just commiserating
 
@DenysSéguret Clippy tends to be a bit bossy. I wonder what's behind the lint review process.
 
Clippy is great but I consider its fixes more like suggestions than rules.
 
I feel that there are too many of them at warning level. For instance, Clippy asks us to also derive Eq when deriving PartialEq when possible. And I say screw that because I don't want my APIs to commit to that.
Another great example is that recent one telling everyone to use implicit named arguments for formatting macros wherever possible.
 
3:12 PM
Yes. In the precise case of Eq, it's clearly stupid when it's in a program and not a lib
 
Implicit named arguments only exist since 1.58, and frankly there isn't that big a benefit in migrating existing code.
 
I'll probably try once again to convince my colleagues to remove it from the CI
(while it's great to run it from time to time to improve the code)
 
I'd rather not remove it because it may indeed find things worth fixing. My point is that stuff such as the ones above are not.
 
 
1 hour later…
4:18 PM
@DenysSéguret I'd argue it's best to continue running it, however you can use flags easily (in CI) to allow the warnings you don't care about. This way you get the best of both worlds: the warnings you care for are flagged every time, and the ones you don't care for are never flagged.
 
Right now we use #[allow... local directives for the local problems. And we comply with the global requests of Clippy This way we don't have to discuss.
 
 
1 hour later…
5:27 PM
> warning: the following packages contain code that will be rejected by a future version of Rust: ntapi v0.3.7
First time I see something like this in the wild.
 

« first day (3033 days earlier)      last day (455 days later) »