-
-
Notifications
You must be signed in to change notification settings - Fork 522
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
fix winform control complie #420
base: master
Are you sure you want to change the base?
Conversation
Hello, could you please provide test cases which demonstrate the functionality before and after your changes. Could you also describe why you belive the LoadAllDepdencies command is necessary? dnSpy usually loads all depdendecies as needed. |
3.Regarding The "LoadDependencies" |
Link to issue(s) this pull request covers:
Problem
Custom control derived from 'System.Windows.Forms.Control' compilation error
Solution
Modify the judgment condition of 'IsWInForm' to be derived from 'System. Windows. Forms. Control' and contain a field of type 'System. Component. IContainer' with the name 'components'