<blockquote>
<p>The problem with the C++ example is that <code>setter</code>, <code>getter</code>, <code>indexer</code> are types not functions, they should be highlighted as types. Whilst <code>pl::Path::root</code>, pl::Path::segment<code>and</code>pl::Path::scheme` are functions and should be highlighted as such.</p>
</blockquote>

<p>Well, we're again asking what the changes proposed here are supposed to highlight: functions or calls.<br>
IMO highlighting calls is fine, whether it's a constructor or a function, and not highlighting functions/members used as value is fine too.  But depending on what you see in the feature, you could indeed argue only functions should be highlighted, irrespective of whether they are called or not, and constructors shouldn't.</p>

<p>Really, before continuing and showing cases and arguing whether it's right or wrong, we should sort this out.  What is this supposed to do in each one's opinion?</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-247554200">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJxJ6DoZCAIzkMlRIGzJImOoUxDEaks5qql0igaJpZM4J8sOE">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJxgRP4NqUkW2DOR9Ob0juwxRvVr9ks5qql0igaJpZM4J8sOE.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-247554200"></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":"@b4n in #1231: \u003e The problem with the C++ example is that `setter`, `getter`, `indexer` are types not functions, they should be highlighted as types. Whilst `pl::Path::root`, pl::Path::segment` and `pl::Path::scheme` are functions and should be highlighted as such.\r\n\r\nWell, we're again asking what the changes proposed here are supposed to highlight: functions or calls.\r\nIMO highlighting calls is fine, whether it's a constructor or a function, and not highlighting functions/members used as value is fine too.  But depending on what you see in the feature, you could indeed argue only functions should be highlighted, irrespective of whether they are called or not, and constructors shouldn't.\r\n\r\nReally, before continuing and showing cases and arguing whether it's right or wrong, we should sort this out.  What is this supposed to do in each one's opinion?"}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/1231#issuecomment-247554200"}}}</script>