Sublime Forum

Build 3114 - ASP Syntax tweaked, now fixed

#1

After upgrading to Stable 3114, I noticed that the syntax file was incorrectly highlighting “sub” - for instance, in my code:
submitfirstname = rs.fields(“SFIRSTNAME”)
submitlastname = rs.fields(“SLASTNAME”)

The SUB in submit was highlighted as the beginning of a SUB, similar results when a word contains FUNCTION.

I modified ASP.sublime-package/ASP.sublime-syntax:11 -
from:
- match: ‘^\s*((?i:function|sub))\s*’
to:
- match: ‘^\s*((?i:function|sub))\b’

I’m not sure if there’s a github repository that I should submit this to, or if posting it here would be sufficient for the developers to integrate it. At the very least, if others are having this issue, here is now to fix it.

0 Likes
#2

Thanks, I’ll integrate it :slightly_smiling: in fact, it is already fixed in my latest PR. I’ll add a test for it to make sure there are no regressions in future :slight_smile:

0 Likes
#3

For the future: Changes or bugs like these are best submitted to https://github.com/sublimehq/Packages.

0 Likes
#4

Thanks for the quick response, kingkeith, and also thanks for the github pointer, FichteFoll!

0 Likes