Run MilCodeGen for PresentationCore#10434
Conversation
|
This PR is blocked by #10430. |
Contributes to dotnet#10429
8f44ef0 to
8d86bf3
Compare
dipeshmsft
left a comment
There was a problem hiding this comment.
LGTM. No doubt there are spacing issues and in TextDecorationCollection we have [index] converted to [ index ], but I guess it should be handled in a separate PR.
@ThomasGoulet73 are you planning to work on the spacing and other issue ?
|
That is to reduce the diff. |
I thought this was reverted back to this state because of the limitation of the generator. This code gets generated from this place : and if we remove the spaces, the generator crashes throwing a syntax error. |
|
Eitherway is possible tbh, I know where its coming from but I didn't try to modify it. |
|
I did hit the bracket limitation but the main reason I reverted
I hit this problem also but there's way to workaround this limitation, you just need to concat the string with the I do plan on working on formatting the generated code in the future. |
Contributes to #10429
Description
Generates the code in PresentationCore using the changes from #10430.
I recommend reviewing only the last commit and with whitespace changes hidden, use this link.
Customer Impact
None, shouldn't affect customers.
Regression
No.
Testing
Local build + CI + I manually validated the generated code.
Risk
Low, there shouldn't be any functional changes.
Microsoft Reviewers: Open in CodeFlow