-
Notifications
You must be signed in to change notification settings - Fork 478
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
Some cleanup of internals #3711
Conversation
tomlm
commented
Apr 9, 2020
- Add dialogComponentRegistration
- move memoryscope registration to dialogComponentRegistration
- fix pathResolvers to not be order depedent by checking char after alias is valid path
- rename context to sourceContext for sourcemap info to be clearer and to enable changes to sourceContext without introducing breaking changes
- made SourceRange internal
* move memoryscope registration to dialogComponentRegistration * fix pathResolvers to not be order depedent by checking char after alias is valid path * rename context to sourceContext for sourcemap info to be clearer. * made SourceRange internal
@@ -0,0 +1,24 @@ | |||
using System; |
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.
copyright
/// </summary> | ||
public class SourceContext | ||
{ | ||
public SourceContext() |
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.
Any reason to keep the default constructor?
/// <summary> | ||
/// SourceContext used to build sourcemap of metadata about the call relationships between assets. | ||
/// </summary> | ||
public class SourceContext |
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.
This is a better parameter than the stack we were passing around, good call.
@@ -0,0 +1,24 @@ | |||
using System; |
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.
copyright