cullmann closed this revision.
cullmann added a comment.
Git commit e1c3d16a35ddae90de6e457a31f54d899abb84c8 by Christoph Cullmann, on
behalf of Nibaldo González.
Committed on 15/08/2018 at 18:22.
Pushed by cullmann into branch 'master'.
Modelines: add byte-order-mark & small fixes
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
Works, will integrate, thanks for the patch!
REPOSITORY
R216 Syntax Highlighting
BRANCH
arcpatch-D14274
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dh
cullmann added a comment.
Will try that out again.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns,
demsking, cullm
nibags updated this revision to Diff 39789.
nibags added a comment.
- Fix test (use Modelines in Python instead of Bash)
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14274?vs=39717&id=39789
BRANCH
arcpatch-D14274
REVISION DETAIL
https://
cullmann requested changes to this revision.
This revision now requires changes to proceed.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapa
cullmann reopened this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
I needed to revert this again, as we get tests failures:
https://build.kde.org/job/Frameworks%20syntax-highlighting%20kf5-qt5%20SUSEQt5.9/97/testReport/junit/(root)/TestSuite/testhi
This revision was automatically updated to reflect the committed changes.
Closed by commit R216:bffb66cb3ed8: Modelines: add byte-order-mark & small
fixes (authored by nibags, committed by cullmann).
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D1
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
OK, thanks.
REPOSITORY
R216 Syntax Highlighting
BRANCH
arcpatch-D14274
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
nibags updated this revision to Diff 39552.
nibags added a comment.
- Corrects references erroneously included in previous update
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14274?vs=39551&id=39552
BRANCH
arcpatch-D14274
REVISION DETAIL
nibags updated this revision to Diff 39551.
nibags added a comment.
- Add test & add DetectSpace.
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14274?vs=38477&id=39551
BRANCH
fix-modelines
REVISION DETAIL
https://phabricator.kde.org/D1427
nibags added a comment.
I will update the diff including the tests.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns,
nibags added a comment.
So I keep the DetectSpaces rule and only delete the LineContinue.
By having a file with Modelines as a test, randomly, different .html and .ref
files are generated each time I run "testhighlighter_test" and
"htmlhighlighter_test". That is, sometimes the .html and
dhaumann added a comment.
Sidenote: Looking into the code in KTextEditor, there is:
} else if (var == QLatin1String("bom") || var ==
QLatin1String("byte-order-mark") || var == QLatin1String("byte-order-marker")) {
if (checkBoolValue(val, &state)) {
m_config->setBom(
cullmann requested changes to this revision.
cullmann added a comment.
This revision now requires changes to proceed.
is only a performance optimization, as it just skips over
whitespaces before any of the costly rules are tried to match. I think that
should just be added again.
Otherwise,
nibags updated this revision to Diff 38477.
nibags added a comment.
- Delete update references
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14274?vs=38201&id=38477
BRANCH
fix-modelines
REVISION DETAIL
https://phabricator.kde.org/D14274
nibags edited the summary of this revision.
nibags added reviewers: turbov, dhaumann, cullmann, vkrause, Kate.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D14274
To: nibags, turbov, dhaumann, cullmann, vkrause, #kate
Cc: kwrite-devel, kde-frameworks-devel,
nibags created this revision.
Restricted Application added projects: Kate, Frameworks.
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel.
nibags requested review of this revision.
REVISION SUMMARY
Changes:
- Add missing keyword: `byte-order-mark` (there is only "b
17 matches
Mail list logo
