-
Notifications
You must be signed in to change notification settings - Fork 118
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
Write basic parser for comment references #2634
Conversation
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.
Excellent! Impl and tests lgtm
if (prefixResult.type == _PrefixResultType.parsedConstructorHint) { | ||
children.add(prefixResult.node); | ||
} | ||
// junk / missing we can skip. |
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.
I wasn't sure what this comment is referring to, until I read further. Maybe call out _PrefixResultType.junk
and _PrefixResultType.missing
explicitly.
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.
done
return; | ||
} | ||
|
||
/// Returns true if we started with startChar and ended with endChar |
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.
can reference startChar as [startChar]
and endChar with [endChar]
:D
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.
done
|
||
/// Returns true if we started with startChar and ended with endChar | ||
/// with a matching number of braces. | ||
/// Returns _index to start if returning false. |
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.
"startIndex" would be more meaningful than "start" here.
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.
rephrased
This is intended to replace the string splits/joins and regular expressions scattered throughout markdown_processor.dart in the new lookup system.