dlevin created this revision.
dlevin added a reviewer: cullmann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
dlevin requested review of this revision.
REVISION SUMMARY
Fixed small issues for rvalue highlighting:
- Unary numbers, pre
dlevin added a comment.
Thanks for cleaning up the tests after me!
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D19198
To: dlevin, cullmann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, ngraham, bruns, demsking,
cullmann, sars, dhaumann
dlevin created this revision.
dlevin added a reviewer: Framework: Syntax Highlighting.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
dlevin requested review of this revision.
REVISION SUMMARY
Accurate implementation of BrightScript languag
dlevin updated this revision to Diff 44885.
dlevin added a comment.
Changed syntax license to MIT.
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D16677?vs=44881&id=44885
REVISION DETAIL
https://phabricator.kde.org/D16677
AFFECTED FILES
aut
dlevin added a comment.
There are various reasons I was using 1-element keyword lists instead of
StringDetect:
1. keyword properly recognizes value surrounded by non-word characters,
StringDetect does not.
2. BrightScript language is case insensitive, adding insensitive="true" to
ever
dlevin added a comment.
Just double checking whether any additional actions required from my side or
'landing' a patch means it will be a part of the next release automatically.
If so then where I could find a release roadmap?
Thanks in advance!
REPOSITORY
R216 Syntax Highlighting
REVI
dlevin added a comment.
I do not have a KDE commit account, so I guess I need your help to integrate
this patch.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D16677
To: dlevin, #framework_syntax_highlighting, cullmann
Cc: dhaumann, cullmann, kwrite-dev
dlevin added a comment.
Thanks a lot for helping me with this! One last question: how do I know that
patch was finally merged, by periodically checking the github mirror git
repository? https://github.com/KDE/syntax-highlighting.git
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
ht
dlevin added a comment.
@ngraham, so I have to match commit authors in syntax-highlighting Git
project against that list? Curious whether all those people have rights to push
patches particularly into this project. Still it feels like this is not an easy
task to do that manually, so I would
dlevin added a comment.
You can find my credentials at the beginning of the file I added in this
patch: data/syntax/brightscript.xml
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D16677
To: dlevin, #framework_syntax_highlighting, cullmann
Cc: ngraham, d
dlevin updated this revision to Diff 45260.
dlevin added a comment.
Replaced missing "code" attribute with "g".
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D16677?vs=44885&id=45260
REVISION DETAIL
https://phabricator.kde.org/D16677
AFFECTED FILES
autotests/html/brightscript.b
dlevin added a comment.
Thanks for catching this. Did re-run tests locally, seems no other issue
remaining. Please check.
REVISION DETAIL
https://phabricator.kde.org/D16677
To: dlevin, #framework_syntax_highlighting, cullmann
Cc: ngraham, dhaumann, cullmann, kwrite-devel, kde-frameworks-de
dlevin created this revision.
dlevin added a reviewer: cullmann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
dlevin requested review of this revision.
REVISION SUMMARY
Subs and functions could be unnamed in closures.
REPOSITORY
R216 S
dlevin updated this revision to Diff 46371.
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17204?vs=46369&id=46371
REVISION DETAIL
https://phabricator.kde.org/D17204
AFFECTED FILES
autotests/html/brightscript.brs.html
autotests/input/brights
dlevin added a comment.
Thanks for the review. Btw, is there any chance to preserve my name/email in
the final commit?
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D17204
To: dlevin, cullmann
Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngra
dlevin added a comment.
This language does not have folding test. I did not provide validation for it
before. Should I do that?
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D17204
To: dlevin, cullmann
Cc: dfaure, kwrite-devel, kde-frameworks-devel, has
dlevin created this revision.
dlevin added a reviewer: cullmann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
dlevin requested review of this revision.
REVISION SUMMARY
Global and local function folding, including closures.
REPOSITORY
dlevin updated this revision to Diff 46667.
dlevin added a comment.
Increased version number.
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17295?vs=4&id=46667
REVISION DETAIL
https://phabricator.kde.org/D17295
AFFECTED FILES
autotest
dlevin added a comment.
Thanks for the fix, yes it is valid. Previously "end" was used as a
standalone keyword, but keyword requires space after the word. Now it used in
StringDetect to detect cases like "end sub" and "endsub" in the same rule.
REPOSITORY
R216 Syntax Highlighting
REVISION
dlevin created this revision.
dlevin added a reviewer: cullmann.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
dlevin requested review of this revision.
REVISION SUMMARY
QtCreator requires that keyword should be wrapped with white spaces o
dlevin added a comment.
Yeah, this might be another bug in the QtCreator (already fixed one crash in
their highlighting implementation). They asked me to not put much effort
because that code it deprecated anyway in favor or KDE lib. But they just
released Creator 4.8.0 and I do not expect a
21 matches
Mail list logo