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

Fix inconsistencies in order of GeneratedDllImport fields #60049

Conversation

elinor-fung
Copy link
Member

The most common ordering I saw for existing DllImports based on some simple regex searches was EntryPoint and then remaining ones in alphabetical order, so I went with that.

cc @AaronRobinsonMSFT @jkoritzinsky

@jkoritzinsky
Copy link
Member

We should update our code-fix to generate the properties in the order that matches this do we stay consistent.

@stephentoub
Copy link
Member

(And, very low priority, consider a style analyzer that suggests an ordering, ala https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0036)

@elinor-fung elinor-fung merged commit d81945e into dotnet:feature/use-dllimport-generator Oct 6, 2021
@elinor-fung elinor-fung deleted the inconsistentAttrOrder branch October 6, 2021 14:47
@ghost ghost locked as resolved and limited conversation to collaborators Nov 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants