[Geany-devel] [geany/geany] 795ee4: Merge pull request #28 from RetroX/patch-1

Lex Trotman elextr at xxxxx
Sat Feb 25 08:55:57 UTC 2012


On 25 February 2012 19:35, Frank Lanitz <frank at frank.uvena.de> wrote:
> On Sat, 25 Feb 2012 10:43:10 +1100
> Lex Trotman <elextr at gmail.com> wrote:
>
>> On 25 February 2012 09:44, Thomas Martitz
>> <thomas.martitz at student.htw-berlin.de> wrote:
>> > Am 24.02.2012 23:34, schrieb Lex Trotman:
>> >
>> >> I don't agree with this change, the types are just that, types, not
>> >> keywords, they should not be highlighted as keywords.  They are not
>> >> always available.  This change should be reverted.
>> >>
>> >
>> > The list contained types before the commit and the commit just
>> > added more for completeness.
>> >
>> > I suggest using the secondary for types instead.
>>
>> Yes, good idea, if I understand the comment in
>> highlightingmappings.h:206 it is meant for types.
>>
>> Cheers
>> Lex
>>
>> PS the existing list contains the fundamental types that are always
>> available without headers, these new ones need a header (though size_t
>> is used so much that almost any header will do :)
>
>
> I did like that idea adding more often used one. But you are right
> cleaning up a bit and maybe resorting them would be might a ice idea.
>
> Cheers,
> Frank

Hi Frank,

I think Thomas' idea of adding those that are not fundamental types to
the secondary list is the right thing, they get highlighted as types
not keywords and as you say the common ones are then available.  Best
of both worlds :).

I think all the ones that were originally in the list were fundamental
in C++11, so its only the new ones IIUC.

I was going to do it today but havn't had time.

Cheers
Lex

>
>
> --
> http://frank.uvena.de/en/
>
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> https://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
>



More information about the Devel mailing list