This repository has been archived by the owner on Mar 8, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 13
[bug] Missed token for regular expressions #22
Labels
Comments
Blocked by bblfsh/sdk#318 |
36 tasks
Can confirm that it's related to an issue above. But if we can solve it right now if we duplicate some information in the node. For the context, the native AST node contains the following fields:
Right now we set Instead, I will set the |
dennwc
pushed a commit
to dennwc/javascript-driver
that referenced
this issue
Jan 3, 2019
Signed-off-by: Denys Smirnov <denys@sourced.tech>
dennwc
pushed a commit
to dennwc/javascript-driver
that referenced
this issue
Jan 3, 2019
Signed-off-by: Denys Smirnov <denys@sourced.tech>
dennwc
pushed a commit
to dennwc/javascript-driver
that referenced
this issue
Jan 22, 2019
Signed-off-by: Denys Smirnov <denys@sourced.tech>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hi,
I found that token information for node with regexp is missed.
Code:
Let's visualize code without nodes with tokens:
If we check the node with
/\<img/g
we will see:I think this node should contain token with
/\<img
.The text was updated successfully, but these errors were encountered: