Skip to content
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

Code review of SystemSettings code #29

Open
christopher-rtf opened this issue May 11, 2020 · 0 comments
Open

Code review of SystemSettings code #29

christopher-rtf opened this issue May 11, 2020 · 0 comments
Assignees

Comments

@christopher-rtf
Copy link
Contributor

Background: MorphicLiteClientWindows is currently using C# code copied or ported from Morphic Classic for getting/setting system/app settings via the Windows SystemSettings infrastructure. This feature is poorly/not documented in Windows, and James indicated that the Morphic Classic team moved away from the C# implementation due to quality concerns.

Task: Analyze the SystemSettings code (by inspecting/introspecting into Windows 10) using tools such as Process Monitor, OleView, Ghidra, etc. Note any inconsistencies between our C# implementation details and inspected Windows implementation details, or any concerns generally.

Based on the MVP10 timeline (beta on 15-May-2020 and RTM on 1-Jun-2020) and because our project direction is to not rewrite the SystemSettings code for MVP10, this analysis is expected to be partial and not full in nature. We should also strongly consider a full analysis.

@christopher-rtf christopher-rtf self-assigned this May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant