Review Request/New Framework: KF5::SyntaxHighlighting

Dominik Haumann dhaumann at kde.org
Sat Oct 1 19:28:54 UTC 2016


On Fri, Sep 30, 2016 at 10:08 AM, David Faure <faure at kde.org> wrote:
> On samedi 10 septembre 2016 17:47:00 CEST Volker Krause wrote:
>> please review KF5::SyntaxHighlighting (syntax-highlighting in Git) for
>> becoming a framework :)
>
> Looks good. I found a few things though.
> [...]

Christoph and me were just talking about the release again, and would
like to delay until next month (i.e. KF 5.28). Then, we have a bit
more time to make sure the API ships what KTextEditor requires.

While the highlighting API already looks really good and the existing
code is in excellent (!) shape, the xml definition files also provide
code folding information. And this this part was not much discussed so
far, meaning that we probably need to change API, or at least extend
the API again potentially with virtual functions. We are sorry we
missed this so far.

Hope this is OK?

@Volker: We could release a version without installing the headers for
AbstractHighlighter, but that again would limit the use for other
projects. Or what do you think?

@David: Can you take care of not releasing with KF 5.27?

Btw, say we do not release the framework with KF 5.27, can KTextEditor
already depend on in it master starting next week or so for KF 5.28 ?

Best,
Dominik


More information about the Kde-frameworks-devel mailing list