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

Move all PREFIX_* constants that have more than 10 members to their own class named PREFIX #247

Merged
merged 1 commit into from
Nov 15, 2021

Conversation

tannergooding
Copy link
Member

@tannergooding tannergooding commented Nov 14, 2021

With this (And the related past couple PRs splitting out "constants"), the follow approx changes exist:

Interop:  11592 Types; 36997 Constants; 6683 Guids; 5721 method imports; 3266 other public static defines (methods/properties)
--------------------------------------------------------------------------------------------------------------------------------
DirectX:   2277 Types;   207 Constants;   12 Guids;  117 method imports;  172 other public static defines (methods/properties)
Windows:   9707 Types;  4445 Constants;  228 Guids; 5511 method imports; 2199 other public static defines (methods/properties)
WinRT:      192 Types;     4 Constants;    0 Guids;   93 method imports;   21 other public static defines (methods/properties)

Where Types is the number of types in the given namespace; the rest are the number of members in the respective static class with the DLL imports. This brings the largest class (static class Windows) down to 12383 members out of the 52667 original members; or ~24% of the original size.

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

Successfully merging this pull request may close these issues.

1 participant