Tweaking things to work with LSL Forge files #17
Conversation
Recognizes .lslp and .lslm files as LSL Syntax highlights $import, $module and //pragma inline
|
I will pull this down and test it out tonight and get it merged. Thanks for the contribution! |
|
cool. i've made a package to hook into LSL Forge from atom since eclipse is such total bloatware. let me know if you're interested. |
| @@ -61,6 +61,18 @@ | |||
| 'name': 'punctuation.definition.comment.lsl' | |||
| 'match': '(//).*$\\n?' | |||
| 'name': 'comment.line.double-slash.lsl' | |||
| 'pragma_inline': | |||
There was a problem hiding this comment.
@furroy I don't understand the changes you propose here.
One the one hand you suggest captures groups for the match for pragma_inline, on the other you assign the whole match a scope name. Additionally your regex doesn't have groups at all, so referencing a captures: 1: ... group doesn't make sense.
Same with modules.
There was a problem hiding this comment.
I just tweaked similar examples I found online and worked for me. Maybe they don't follow some sort of best practices, but I've not had any issues with these changes.
Recognizes .lslp and .lslm files as LSL
Syntax highlights $import, $module and //pragma inline