IDE-569 Syntax highlight KEL #340
Conversation
|
https://track.hpccsystems.com/browse/IDE-569 |
dehilsterlexis
force-pushed
the
IDE-569
branch
from
00272fb
to
3de94f1
Compare
|
@GordonSmith I would like to do a walk through the code concerning the use of the boost xml archive for a new file for sample language text. The code is commented out but can be easily tested. It should work but is not. I would prefer to use the file instead of hard coding the text. I will highlight the text in a comment. |
|
|
||
| if (m_langRef) | ||
| { | ||
| //m_langRef->loadSamples(); |
There was a problem hiding this comment.
@GordonSmith here is the boost archive load I have commented out on line 1416.
dehilsterlexis
force-pushed
the
IDE-569
branch
from
3de94f1
to
e33f6e2
Compare
| @@ -0,0 +1,173 @@ | |||
| // Scintilla source code edit control | |||
| /** @file LexKEL.cxx | |||
There was a problem hiding this comment.
Comment looks wrong - I would remove it in its entirety
| 0, | ||
| }; | ||
|
|
||
| LexerModule lmGENERAL(SCLEX_GENERAL, ColouriseGENERALDocSensitive, "general", FoldGENERALDoc, generalWordLists); |
There was a problem hiding this comment.
What is the goal of the General lexer?
There was a problem hiding this comment.
The general lexer is for non-language related color selections. I tried coding the "general" color settings without the lexer. After a 1 day trip down that road without success and everything broken beyond repair and only getting worse, I decided the easiest way to implement the non-language specific color preferences was to create a "general" lexer. It solved that problem quite nicely.
| ** Lexer for C++, C, Java, and JavaScript. | ||
| **/ | ||
| // Copyright 1998-2005 by Neil Hodgson <neilh@scintilla.org> | ||
| // The License.txt file describes the conditions under which this software may be distributed. |
| @@ -537,6 +537,7 @@ | |||
| #define IDC_STATIC_WUARGUMENTS 524 | |||
| #define IDC_COMBO1 524 | |||
| #define IDC_COMBO_MODULES 524 | |||
| #define IDC_COMBO_ELEMENT_TYPE 524 | |||
There was a problem hiding this comment.
This was auto-generated by VisualStudio. I did it via the GUI editor.
There was a problem hiding this comment.
I think that is fine, as long as none of those IDs clash on the window in question. But in general you can go in and tidy them up by hand.
Fixed IDE-569 Signed-off-by: dehilsterlexis <david.dehilster@lexisnexis.com>
dehilsterlexis
force-pushed
the
IDE-569
branch
from
e33f6e2
to
e19f05a
Compare
|
@GordonSmith fixed the comments in the Lexer files and answered your questions. |
|
@richardkchapman This is ok to pull (into master) |
Fixed IDE-569
Signed-off-by: dehilsterlexis david.dehilster@lexisnexis.com