martonmiklos created this revision.
martonmiklos added a reviewer: Framework: Syntax Highlighting.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
martonmiklos requested review of this revision.
REVISION SUMMARY
Add FreeCAD FCMacro extension
martonmiklos updated this revision to Diff 38273.
martonmiklos added a comment.
Updated revision to 8 from 7 as suggested in the review.
REPOSITORY
R216 Syntax Highlighting
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D14275?vs=38204&id=38273
REVISION DETAIL
https://phabricato
martonmiklos created this revision.
martonmiklos added a reviewer: Framework: Syntax Highlighting.
martonmiklos added a project: Framework: Syntax Highlighting.
Restricted Application added projects: Kate, Frameworks.
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel.
mar
martonmiklos created this revision.
martonmiklos added a reviewer: Framework: Syntax Highlighting.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
martonmiklos requested review of this revision.
REVISION SUMMARY
ctp extension is used by
martonmiklos added a comment.
In https://phabricator.kde.org/D7367#143391, @cullmann wrote:
> Should be fixed now:
>
> commit
https://phabricator.kde.org/R216:fd79b6842c6f33a1aca4f500354ca2a69c52689e
> Author: Christoph Cullmann
> Date: Wed Sep 6 08:08:49 2017 +0200
>
>
martonmiklos added a comment.
In https://phabricator.kde.org/D7367#143314, @cullmann wrote:
> syntax-highlighting.git, on first glance not landed atm.
>
> Can you commit on your own or should I push it for you Miklos?
Hi guys,
I am on holiday ATM so if you could push it that
martonmiklos added a comment.
In https://phabricator.kde.org/D7367#138232, @cullmann wrote:
> Hmm, the diff still looks like it has the old names, perhaps the update
went wrong somehow?
Yepp, I have missed the second page after the diff upload. It should be
correct now.
REVISION
martonmiklos updated this revision to Diff 18514.
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7367?vs=18310&id=18514
REVISION DETAIL
https://phabricator.kde.org/D7367
AFFECTED FILES
autotests/html/highlight.hex.html
autotests/input/highlight.hex
data/syntax/intelhex.xml
To:
martonmiklos marked an inline comment as done.
martonmiklos added a comment.
In https://phabricator.kde.org/D7367#138030, @cullmann wrote:
> I am happy beside one thing I missed: I think the file should be called
intelhex.xml, given its for the intel hex format only (the name is already
martonmiklos updated this revision to Diff 18310.
martonmiklos added a comment.
Okay so:
- License updated to MIT
- Added detection for invalid RecordType (05<)
- Changed formatting to use builtin formats
- Added autotest case with a sample file with two malformed lines and a HTML
o
martonmiklos added a comment.
In https://phabricator.kde.org/D7367#136687, @cullmann wrote:
> Could you add a minimal example as unit test?
Sure, I have just seen that these xmls could be autotested.
> And could we have as license something like "MIT"?
Sure, I have no obj
martonmiklos created this revision.
martonmiklos added a project: Framework: Syntax Hightlighting.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
I have created an syntax highlight file for the Kate.
The coloring s
12 matches
Mail list logo