-
Notifications
You must be signed in to change notification settings - Fork 498
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor the registry-relevant functions. #2490
Conversation
The right click menu register functions are refactored with `windows` crate. The refactored code seems simpler and easier to be read and maintained.
I think this commit could be a good start to migrate from |
I think it would be worth having another go trying the It's new, so there's a small possibility that it is broken, but I don't see any bug reports about it, so I think that chance is very low. It's not very widely used, but still there's 2000 downloads the past 90 days, so I would think that someone else has encountered issues. |
Hi @fredizzimo , I've finished changing the logics. |
Thank you, this did indeed simplify the code a lot! |
The right click menu register functions are refactored with
windows
andwindows-registry
crates. The refactored code seems simpler and easier to be read and maintained.What kind of change does this PR introduce?
Did this PR introduce a breaking change?