<blockquote>
<blockquote>
<p>isIgnoreToken() works differently than in uctags.</p>
</blockquote>
</blockquote>
<blockquote>
<p>Does it? Isn't just the API different?</p>
</blockquote>
<p>One difference is in how isIgnoreToken() is used in c.c. In uctags parsers (or c.c in particular) don't call this function at all - this function is just called in main and based on it some tags are filtered-out. However, in Geany's c.c this function helps to the parser to determine what to do - if the ignored token is e.g. a macro with parameters, everything is skipped until ")".</p>
<p>This also translates to the ignore file format - in the Geany version there's the "+" which indicates that parentheses should be skipped and "=" which specifies with what the token should be replaced. There's nothing like that in uctags.</p>
<p>Personally I'd be for removing this functionality as if we want to preserve it, we'd have to add it to the new cpp uctags parser which doesn't use this. As I said, I don't use the ignored file myself so I'm not the right guy to ask whether it's something important for others.</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/pull/1263#issuecomment-279719773">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ0JKVUG-3AiHqNUL3CmlRci8OEGmks5rcbisgaJpZM4KXvBh">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7bcyfrinOOuo7texNyUQGJUTI2Bks5rcbisgaJpZM4KXvBh.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/pull/1263#issuecomment-279719773"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request 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":"@techee in #1263: \u003e\u003e isIgnoreToken() works differently than in uctags.\r\n\r\n\u003e Does it? Isn't just the API different?\r\n\r\nOne difference is in how isIgnoreToken() is used in c.c. In uctags parsers (or c.c in particular) don't call this function at all - this function is just called in main and based on it some tags are filtered-out. However, in Geany's c.c this function helps to the parser to determine what to do - if the ignored token is e.g. a macro with parameters, everything is skipped until \")\".\r\n\r\nThis also translates to the ignore file format - in the Geany version there's the \"+\" which indicates that parentheses should be skipped and \"=\" which specifies with what the token should be replaced. There's nothing like that in uctags.\r\n\r\nPersonally I'd be for removing this functionality as if we want to preserve it, we'd have to add it to the new cpp uctags parser which doesn't use this. As I said, I don't use the ignored file myself so I'm not the right guy to ask whether it's something important for others."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1263#issuecomment-279719773"}}}</script>