seti-syntax space between function name and bracket

screenshot from 2014-10-29 23 21 53

In C++, if there is a space between function name and bracket, then two brackets' color will be different. Will it be solve or not

Oct 29 '14 15:10 wchichih

This should be an easy fix. Hopefully the team picks it up and patches it soon. +1

Oct 30 '14 14:10 karloluis

So the issue seems to be that where there's a space between the if and the paren, atom wrap it in a span with additional classes "punctuation definition parameters c":

<span class="punctuation definition parameters c”>(</span>

here's the if block with no space:

<span class="source c++"><span class="meta block c"><span class="leading-whitespace indent-guide">  </span>
<span class="keyword control c">if</span>(k === w-&gt;key) <span class="meta block c"><span class="punctuation section block begin c">{</span></span>
</span>
</span>

and with the space:

<span class="source c++"><span class="meta block c"><span class="leading-whitespace indent-guide">  </span><span class="keyword control c">if</span><span class="meta initialization c"> <span class="punctuation definition parameters c">(</span></span>k
=== w-&gt;key) <span class="meta block c"><span class="punctuation section block begin c">{</span></span>
</span>
</span>

I'm not actually sure how best to resolve this. It's the ".punctuation.definition.parameters" class that adds the blue color, which is there with a space, and not otherwise. If this isn't correct behavior, I would file a bug in the Atom repo and see what those guys think.

However, if there should be a differentiation between space and not and it's it's just weird to have them different colors I can push a fix to make .punctuation.definition.parameters white in c.

@alim0x @karloluis: thoughts?