search

overdrivenpotato / rust-psp

Rust on PSP. Panic and allocation support. Access PSP system libraries.
Other
588 stars 31 forks source link

chore(cargo-psp): update deps + use clap::Parser #162

Closed SK83RJOSH closed 5 months ago

SK83RJOSH commented 5 months ago

Decided to fix up warnings in cargo-psp, as well as update all it's dependencies. As a bonus removed a bunch of clap boilerplate using it's derive feature. 🙂

SK83RJOSH commented 5 months ago

In general I'm concerned that we may accidentally lose functionality by switching to the derive feature of clap, but if you can prove otherwise, the rest looks good and I'm happy to accept this.

Anything in particular you'd like me to test? mksfo was the most complex case, and appears to be working just fine from my testing. The rest were relatively straight-forward, with the exception of pack-pbp, since it used NULL instead of optional params. I special cased that one (personally, would prefer to used named optional arguments... but that would have been a breaking change 🙂)