ktexteditor and syntax-highlighting porting

Dr.-Ing. Christoph Cullmann cullmann at absint.com
Sun Jul 29 11:18:36 BST 2018


Hi,

> On Sun, Jul 29, 2018 at 9:53 AM, Dominik Haumann <dhaumann at kde.org> wrote:
>> [...]
>>
>> What we can add:
>> 1. QHash<uint16_t, Format> Definition::formatHash() const;
>> 2. Format Definition::format(uint16_t) const;
>> 3. QHash<uint16_t, Format> Repository::formatHash() const;
>> 4. Format Repository::format(uint16_t) const;
> 
> I see an issue with 3) because Definitions are lazy-loaded.
> So the number of Format items may grow over time if more Definitions are loaded.
> So I think we should not add this.
as commented in the phabricator request: I would actually not do repository global id's at all.
Or is that needed for embedded highlightings?

Greetings
Christoph

-- 
----------------------------- Dr.-Ing. Christoph Cullmann ---------
AbsInt Angewandte Informatik GmbH      Email: cullmann at AbsInt.com
Science Park 1                         Tel:   +49-681-38360-22
66123 Saarbrücken                      Fax:   +49-681-38360-20
GERMANY                                WWW:   http://www.AbsInt.com
--------------------------------------------------------------------
Geschäftsführung: Dr.-Ing. Christian Ferdinand
Eingetragen im Handelsregister des Amtsgerichts Saarbrücken, HRB 11234


More information about the KWrite-Devel mailing list