Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Perl highlighting issue in regexp #1029

Closed
mnowaksuse opened this issue Jun 12, 2017 · 2 comments · Fixed by #2048
Closed

Perl highlighting issue in regexp #1029

mnowaksuse opened this issue Jun 12, 2017 · 2 comments · Fixed by #2048

Comments

@mnowaksuse
Copy link

r at the end of the regexp is causing following text to wrongly highlight. If removed, it's OK.

if (get_var('REPO_0')) {
     my $sles_debug_repo = get_var('REPO_0') =~ s/(Server|Desktop)/\U$1\E/r;
     $sles_debug_repo =~ s/DVD/POOL/;

image
@keith-hall keith-hall mentioned this issue Jun 14, 2017
Closed
@gullevek
Copy link

I see the same issue. It seems that /r is not supported in the regex highlighter.

screen shot 2017-11-24 at 14 12 12

TheClams pushed a commit to TheClams/Packages that referenced this issue May 10, 2018
@Clamsax
[Perl] Fix a few regexp related issues and
7c1e8be 
Fix issue sublimehq#541 with  space at the beginnign of a regexp
Fix issue sublimehq#1495 and sublimehq#1594 where // can break the syntax highlighting
Fix issue sublimehq#1325 and sublimehq#1029 related to regexp modifier

Fix issue sublimehq#1326 with variable starting with underscore
@TheClams TheClams mentioned this issue May 10, 2018
Closed
deathaxe added a commit to deathaxe/sublime-packages that referenced this issue May 11, 2018
@deathaxe
[Perl] Refactor whole syntax definition to drop Oniguruma
8293bad 
Fixes sublimehq#541, sublimehq#1029, sublimehq#1325, sublimehq#1326, sublimehq#1495

Supersedes sublimehq#1556

According to sublimehq#481 this commit provides an Oniguruma free Perl syntax
definition written from scratch.

OUTLINE

  1. fixes all currently open issues
  2. much more feature complete
  3. completely compatible with new regex engine
  4. uses embed to include custom syntaxes within
     POD and HEREDOCS (CSS,JS,JSON,HTML,SQL,XML)
  5. provides a syntax test file
  6. uses as much atomic rules from Perl.sublime-syntax and
     ModernPerl package as possible (with some fixes/modifications)
  7. follows the doc http://perldoc.perl.org as close as possible
  8. add some snippets with some files renamed to match naming style

REMARKS

1. The benchmark tests on a 100k file with all kinds of statements
   output about 360ms for evaluation on a Windows box.

2. The main focus is to create a robust base for syntax highlighting
   rather than scoping each language entity/construct in detail.

   a) Not all meta scoping RFCs are implemented in detail due to
      some limitations caused by HEREDOC. Especially pushing in/out
      of `meta.block` or `meta.braces` breaks some HEREDOC constructs.

   b) Scope names were changed to follow the scope naming guidelines
      as close as possible and try to use as general names as possible.
      -> Maybe needs some refines?

3. At the current point the implementation is a work in progress and
   needs testing by perl power users to

   a) check for highlighting to work in all situations

   b) constants/variables/functions are scoped correctly

4. This commit adds some snippets but does not include any completions
   in favor of https://packagecontrol.io/packages/Perl%20Completions
@deathaxe deathaxe mentioned this issue May 11, 2018
Merged
@deathaxe
Copy link
Collaborator

deathaxe commented Dec 4, 2018

@wbond: fixed by #1564

@deathaxe deathaxe mentioned this issue Aug 14, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Perl] Feature Update 2019 deathaxe/sublime-packages
3 participants
@gullevek @mnowaksuse @deathaxe