-
Notifications
You must be signed in to change notification settings - Fork 510
Conversation
Co-authored-by: Lo茂c Sharma <sharma.loic@gmail.com>
Co-authored-by: Lo茂c Sharma <sharma.loic@gmail.com>
Co-authored-by: Lo茂c Sharma <sharma.loic@gmail.com>
@loic-sharma my understanding that parameters is just parameters. Can be generic or not. At least this is how I understand the code |
Co-authored-by: Lo茂c Sharma <sharma.loic@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, than you! 馃
Can you also add a ## Providing hints to the compiler externally ##
to the reflection-in-aot-mode.md document that will point to this one?
<Assembly Name="System.Private.CoreLib"> | ||
<Type Name="System.Collections.Generic.Dictionary`2[[System.Int32,System.Private.CoreLib],[System.String,System.Private.CoreLib]]"> | ||
<Method Name="EnsureCapacity"> | ||
<Parameter Name="System.Int32, System.Private.CoreLib" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add a comment here that the Parameter
is optional and it's only used to disambiguate between overloads if there are any
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add mention that Parameter
is optional.
Co-authored-by: Michal Strehovsk媒 <MichalStrehovsky@users.noreply.github.com>
Co-authored-by: Michal Strehovsk媒 <MichalStrehovsky@users.noreply.github.com>
optional Parameter directive
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
This documentation provide examples of how rd.xml can be presented.
The only reason for this PR is #8194 (comment)
Hopefully @MichalStrehovsky would keep his word 馃槈