8

Add `needless_late_init` lint by Alexendoo · Pull Request #7995 · rust-lang/rust...

 2 years ago
source link: https://github.com/rust-lang/rust-clippy/pull/7995
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

|

LL | };

| +

Copy link

Member

Author

@Alexendoo Alexendoo 25 days ago

This overlaps a little with useless_let_if_seq, needless_late_init doesn't do as many clever transformations as useless_let_if_seq however, hopefully meaning it has less false positives

I wasn't sure what to choose here, I could remove the matching of ifs from needless_late_init

Copy link

Collaborator

@giraffate giraffate 24 days ago

This overlaps a little with useless_let_if_seq, needless_late_init doesn't do as many clever transformations as useless_let_if_seq however, hopefully meaning it has less false positives

Yes, it looks good.

I wasn't sure what to choose here, I could remove the matching of ifs from needless_late_init

IMO it looks good as it is.


About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK