-
Notifications
You must be signed in to change notification settings - Fork 12
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
Highlighting issue with quoted string literal #18
Comments
The ocaml.tmbundle.1.diffdiff --git a/Syntaxes/OCaml.plist b/Syntaxes/OCaml.plist
index c28d2ed..6fbc203 100644
--- a/Syntaxes/OCaml.plist
+++ b/Syntaxes/OCaml.plist
@@ -17,6 +17,27 @@
<string>OCaml</string>
<key>patterns</key>
<array>
+ <dict>
+ <key>name</key><string>string.interpolated.ocaml</string>
+ <key>begin</key><string>\\|</string>
+ <key>end</key><string>\\|</string>
+ <key>beginCaptures</key>
+ <dict>
+ <key>0</key>
+ <dict>
+ <key>name</key>
+ <string>punctuation.definition.string.begin.ocaml</string>
+ </dict>
+ </dict>
+ <key>endCaptures</key>
+ <dict>
+ <key>0</key>
+ <dict>
+ <key>name</key>
+ <string>punctuation.definition.string.end.ocaml</string>
+ </dict>
+ </dict>
+ </dict>
<dict>
<key>captures</key>
<dict> (I don't know a single thing about OCaml, and it probably shows…) |
@Alhadis Yes that is exactly it! |
In fact, there are some pull requests implementing this already: https://github.com/textmate/ocaml.tmbundle/pull/17/files It's just that the activity on this repo is non-existent. |
Describe the bug
The following OCaml code causes a highlighting leak:
Expected behaviour
The string escape characters
{| |}
shouldn't be interfered by a following$
.Related discussion
Additional notes
The text was updated successfully, but these errors were encountered: