n4r1b / ferrisetw

Basically a KrabsETW rip-off written in Rust
Other
65 stars 24 forks source link

Turbofish operator for try_parse #68

Closed daladim closed 2 years ago

daladim commented 2 years ago

This PR depends on #63 and #65 , as its first commits come from them.

This makes it possible to use the turbofish operator for try_parse, which is slightly more convenient than the current solution. This improvement was already considered by @n4r1b quite a long time ago, since there was a // TODO: Find a way to use turbofish operator

daladim commented 2 years ago

Build is failing. I probably did not include all the dependant commits after all. I'll rebase after other PR (the ones that are passing) are merged, that will be easier to rebase.

You can consider this PR as a draft for the moment

n4r1b commented 2 years ago

This is beautiful! I was really looking forward to doing this change, thank you very much @daladim 😄🥳. Feel free to merge when ready!

daladim commented 2 years ago

I've rebased to fix the git conflict, without changing anything really. I'm merging