<blockquote>
<ul>
<li>GRegex vs GNU regex - I kept GRegex in lregex.c (but synced all the rest) but we should think whether not to convert back to GNU regex</li>
</ul>
</blockquote>
<p>Well… I'd rather not bundle yet another library, especially as we have the same feature already available in a dependency.</p>
<blockquote>
<ul>
<li>isIgnoreToken() works differently than in uctags.</li>
</ul>
</blockquote>
<p>Does it?  Isn't just the API different?</p>
<blockquote>
<p>If it simply just dropped certain tags, we could move the logic into TM but right now, it helps the C parser with parsing by suggesting whether the contents of braces should be dropped too. I'm not sure if we need to preserve this functionality (never used the ignored.tags file myself) but if we move to the new cxx parser we'd have to keep this diff unless there's some interest for this in uctags.</p>
</blockquote>
<p>Well, the point of ignore tokens is to mitigate the confusion macros can cause on the C parser.  For example, I have a custom ignore.tags with those:</p>
<pre><code>G_GNUC_DEPRECATED_FOR+
G_GNUC_PRINTF+
G_GNUC_SCANF+
G_GNUC_FORMAT+
G_GNUC_*
...
__STL_NOTHROW
__THROW
GIT_EXTERN=extern
cairo_public
GEANY_DEPRECATED
GEANY_PRIVATE_FIELD
GEANY_DEPRECATED_FOR+
</code></pre>
<p>so various source files parse a lot better, because the parser just performs as if those weren't part of the source.  See it as the poor man's C preprocessor.</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-262809493">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4UQQdtCFEClLj9GdfYilMu-RgANks5rBbcsgaJpZM4KXvBh">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ6SWyC8QFJwzcJzzm8XhmLowaSV8ks5rBbcsgaJpZM4KXvBh.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-262809493"></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":"@b4n in #1263: \u003e - GRegex vs GNU regex - I kept GRegex in lregex.c (but synced all the rest) but we should think whether not to convert back to GNU regex\r\n\r\nWell… I'd rather not bundle yet another library, especially as we have the same feature already available in a dependency.\r\n\r\n\u003e - isIgnoreToken() works differently than in uctags.\r\n\r\nDoes it?  Isn't just the API different?\r\n\r\n\u003e If it simply just dropped certain tags, we could move the logic into TM but right now, it helps the C parser with parsing by suggesting whether the contents of braces should be dropped too. I'm not sure if we need to preserve this functionality (never used the ignored.tags file myself) but if we move to the new cxx parser we'd have to keep this diff unless there's some interest for this in uctags.\r\n\r\nWell, the point of ignore tokens is to mitigate the confusion macros can cause on the C parser.  For example, I have a custom ignore.tags with those:\r\n```\r\nG_GNUC_DEPRECATED_FOR+\r\nG_GNUC_PRINTF+\r\nG_GNUC_SCANF+\r\nG_GNUC_FORMAT+\r\nG_GNUC_*\r\n...\r\n__STL_NOTHROW\r\n__THROW\r\nGIT_EXTERN=extern\r\ncairo_public\r\nGEANY_DEPRECATED\r\nGEANY_PRIVATE_FIELD\r\nGEANY_DEPRECATED_FOR+\r\n```\r\nso various source files parse a lot better, because the parser just performs as if those weren't part of the source.  See it as the poor man's C preprocessor."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1263#issuecomment-262809493"}}}</script>