Opened 6 years ago

Closed 3 years ago

#101 closed defect (fixed)

syntax highlighting gone wrong: dor (//) not understood

Reported by: jquelin Owned by: azawawi
Priority: major Milestone: Satisfy all the requirements of PerlMonks
Component: Syntax Highlighting Version: 0.15
Keywords: syntax highlighting, c, scintilla Cc:

Description

the following snippet:

my $foo = $bar ;
say $foo;

colors badly everything after the second slash.
it seems that dor (defined or), introduced in 5.10, is not parsed correctly...

Change History (8)

comment:1 Changed 6 years ago by szabgab

That's not good.
According to http://www.scintilla.org/ScintillaHistory.html support for was added in Scintilla 1.76.
I summarized the state of the versions here wxWidgets.
It seems in order to have this supported we either have to integrate Scintilla 1.76 ourself or help the wxWidgets developers and the wxPerl developers or implement our own Perl 5 syntax highlighting.

comment:2 Changed 6 years ago by szabgab

In 0.16 we added an experimental feature using PPI for syntax highlighting. It is slow but it seems to work.
To turn on the experimental features one needs to add

experimental: 1

to the .padre/config.yml
while padre is NOT running.
Then on the new Experimental menu you can change the highlighting mode.

comment:3 Changed 6 years ago by szabgab

  • Milestone set to Satisfy all the requirements of PerlMonks

comment:4 Changed 4 years ago by zenogantner

  • Keywords syntax highlighting c scintilla added
  • Summary changed from syntax hilighting gone wrong: dor (//) not understood to syntax highlighting gone wrong: dor (//) not understood

comment:5 Changed 3 years ago by azawawi

  • Owner set to azawawi
  • Status changed from new to assigned

comment:6 Changed 3 years ago by azawawi

related to #257

comment:7 Changed 3 years ago by szabgab

  • Component changed from editor to Syntax Highlighting

comment:8 Changed 3 years ago by azawawi

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in Padre 0.86 and Wx::Scintilla. You need to enable feature_wx_scintilla.

Note: See TracTickets for help on using tickets.