On Tue, Apr 6, 2010 at 11:57 AM, <[email protected]> wrote: > from a quick look at your patch it seems that you're highlighting > cv-qualifiers, storage-class-specifiers and a subset of type-specifiers but > i'm wondering why? > > why do you think it is a good idea to highlight "extern", "mutable", > "register" and so on as types? Storage class specifiers auto, register and mutable only ever apply to "variables" and cv-qualifiers even when used on member functions merely qualify "this". Then i've picked those type specifiers that didn't require any semantic interpretation to disambiguate. Seemed somewhat consistent (also in line with what other lexer based syntax highlighters do, more or less).
Now i have to admit including "extern" and "static" was rather arbitrary (and ambiguous). > Yep, we should use the Semantic Highlighter instead. As far as i can see it's only available in trunk and not yet enabled, but that's interesting. > I won't be in the office this week but if you are interested we can continue > this discussion on #qt-creator next week. Excellent. _______________________________________________ Qt-creator mailing list [email protected] http://lists.trolltech.com/mailman/listinfo/qt-creator
