<p>To summarise:</p>

<p>1) it looks ok for C so long as non-call parens in expressions can't be confused with calls.</p>

<p>2) in C++ its possible that there will be too many erroneous highlights due to C++ using the call syntax in many places.</p>

<p>3) I'm not sure if/how well it applies to any of the other languages <code>Lexcpp.cxx</code> highlights.</p>

<p>I would suggest that the best way ahead is to submit your patch to Scintilla and see what Neil says.  If he is happy then its fine.</p>

<p>The above discussion then becomes purely for Geany and which languages to make it visible for by making the SCE_C_FUNC a non-default style.  Irrespective of which choice is made by default (say visible for C and invisible for C++) a user can always change it in their own configuration anyway.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/1231#issuecomment-247307523">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ-7F7X5ceNCrPplKCA3i5kcY1Srcks5qqTGagaJpZM4J8sOE">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ1slpmPXdUQlKbxOnU4I7fsK2K99ks5qqTGagaJpZM4J8sOE.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/issues/1231#issuecomment-247307523"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #1231: To summarise:\r\n\r\n1) it looks ok for C so long as non-call parens in expressions can't be confused with calls.\r\n\r\n2) in C++ its possible that there will be too many erroneous highlights due to C++ using the call syntax in many places.\r\n\r\n3) I'm not sure if/how well it applies to any of the other languages `Lexcpp.cxx` highlights.\r\n\r\nI would suggest that the best way ahead is to submit your patch to Scintilla and see what Neil says.  If he is happy then its fine.\r\n\r\nThe above discussion then becomes purely for Geany and which languages to make it visible for by making the SCE_C_FUNC a non-default style.  Irrespective of which choice is made by default (say visible for C and invisible for C++) a user can always change it in their own configuration anyway."}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/1231#issuecomment-247307523"}}}</script>