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
cullmann added a comment.
Upsa, you are right.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D17204
To: dlevin, cullmann
Cc: dfaure, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns,
demsking, cullmann, sars, dhaumann
dfaure added a comment.
It seems this commit is the reason for the unittest failure below?
https://build.kde.org/job/Frameworks/view/Everything/job/syntax-highlighting/job/kf5-qt5%20SUSEQt5.10/258/testReport/junit/(root)/TestSuite/folding_test/
--- "/home/jenkins/workspace/Framewor
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:c45565159924: BrightScript: Allow sub to be unnamed
(authored by dlevin, committed by cullmann).
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17204?vs
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
Sure :/ I missed to change author last time, I normally am just
patching/landing via arc, but that doesn't always contain that.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
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 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
cullmann requested changes to this revision.
cullmann added a comment.
This revision now requires changes to proceed.
Looks reasonable, but you need to increase the version.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D17204
To: dlevin, cullmann
Cc: kw
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