<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 7, 2016 at 11:14 PM, Lex Trotman <span dir="ltr"><<a href="mailto:elextr@gmail.com" target="_blank">elextr@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 8 January 2016 at 08:00, Jiří Techet <<a href="mailto:techet@gmail.com">techet@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Jan 6, 2016 at 9:23 PM, Thomas Martitz <<a href="mailto:kugel@rockbox.org">kugel@rockbox.org</a>> wrote:<br>
>><br>
>> Am 06.01.2016 um 21:12 schrieb Jiří Techet:<br>
>>><br>
>>><br>
>>><br>
>>>     It's indeed at least interesting to consider, because at least for .h<br>
>>>     headers there really is some mixed stuff all over the place -- even,<br>
>>>     simply look in Scintilla's source tree.<br>
>>><br>
>>><br>
>>> +1 for having the headers parsed/lexed by the C++ parser (with sources it<br>
>>> may be a bit dangerous and typically the sources have the right C++<br>
>>> extension).<br>
>>><br>
>>><br>
>><br>
>> Not replying to Jiří specifically.<br>
>><br>
>> -1. .h is legitimately a C, it's just that many people get it wrong. And I<br>
>> don't want C++ keywords highlighted in C headers while they are not<br>
>> highlighted it C source files. This is just confusing.<br>
><br>
><br>
> I agree with Matthew here - I think the "damage" caused by parsing C headers<br>
> with the C++ parser/lexer is much smaller than vice versa. Actually a few<br>
> months back a user of my ProjectOrganizer plugin wrote me just because of<br>
> that - he had a C++ project with "h" headers and was surprised that tag<br>
> generation didn't work for him.<br>
><br>
> I created (a highly sophisticated) pull request here:<br>
><br>
> <a href="https://github.com/geany/geany/pull/857" rel="noreferrer" target="_blank">https://github.com/geany/geany/pull/857</a><br>
><br>
> Power users can always add *.h back to C types but I think having it in C++<br>
> is a better default.<br>
<br>
</span>As the failing tests show, better have a BIG warning about breaking<br>
change if we do this.<br></blockquote><div><br></div><div>Fixed now. </div><div><br></div><div>Actually the tests "failed" because the tested files were C++ headers and the previously-generated tags files were incorrect because of the used C parser. So yeah, BEWARE USERS, HEADERS MIGHT BE PARSED CORRECTLY :-) </div><div><br></div><div>Jiri</div></div></div></div>