Skip to content

Commit

Permalink
fix: relax requirements on FunC identifiers to (.*?)
Browse files Browse the repository at this point in the history
The regex to approximate actual FunC naming options is gigantic and too
unwieldy to be put here. That's especially true considering that this
will only be used for highlighting and only in a very contained native
function name mapping scenarios.

The similar "include everything" approach was taken previously in
tact.vim:
https://github.com/tact-lang/tact.vim/blob/ddee48224ac89f4aa93f49d4544193ecd6b6948b/syntax/tact.vim#L77-L83
  • Loading branch information
novusnota committed Jul 28, 2024
1 parent f29fb4b commit 9d8e48e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package/Tact.tmLanguage
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@
<key>comment</key>
<string>FunC identifier</string>
<key>match</key>
<string>((?:[a-zA-Z_\'\?!&amp;]|::)(?:[a-zA-Z0-9_\'\?!&amp;]|::)*)</string>
<string>(.*?)</string>
<key>name</key>
<string>entity.name.function.func.tact</string>
</dict>
Expand Down
2 changes: 1 addition & 1 deletion package/Tact.tmLanguage.json
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@
"patterns": [
{
"comment": "FunC identifier",
"match": "((?:[a-zA-Z_\\'\\?!&]|::)(?:[a-zA-Z0-9_\\'\\?!&]|::)*)",
"match": "(.*?)",
"name": "entity.name.function.func.tact"
}
],
Expand Down

0 comments on commit 9d8e48e

Please sign in to comment.