Upgrade Dependencies to Newest Versions#2
Conversation
wallem89
left a comment
There was a problem hiding this comment.
Nice work done @jscatena88 ! I want to use this crate too but ran into the same issues. Some existing once but also some new clippy issues.
I have put them as comments on the existing and new code. Can you fix them?
I am also to get a hold of @marcelbuesing to discus maintenance of this crate because I would love to help out here!
|
Hey sorry for the delay, I had put this down and was busy with other stuff. I originally was just trying to make the absolute minimal changes possible since I assumed that'd make it easier to review or change. I also didn't want to be opinionated about someone elses work with respect to unused code. But if you are going to be taking over and are cool with it I'm more than happy to be a bit more aggressive. I just went through and tackled all the clippy lints. I also temporarily toggled on clippy pedantic locally and refactored some as casts to hopefully be more robust (kind of odd that can_dbc uses Also all the unused code in candump_parser.rs led me to realize that essentially nothing was in the But I know that is pretty aggressive so I'm open to push back on that. I got the vibe though that you are trying to really clean house |
I took a crack at upgrading all dependencies to their newest versions. I have not yet tested this on real hardware with real DBC files, I'll try to get to that in the next few days and report back. In the meantime I figured I would throw this up for comment.