<blockquote>
<p>We could have a patch file similar to Scintilla which introduces the changes we need in ctags</p>
</blockquote>

<p>Totally, and I support the end target of your changes, just not the process of getting there <g-emoji alias="smile" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png">😄</g-emoji></p>

<p>I accidentally selected the "files changed" tab and it locked the browser for a time and used 1GB of memory  +5328 line -5619 lines.  IMNSHO its not a responsible thing to propose or to accept such a change.  I greatly admire the quality of your submissions, but even you can't be sure whats in such a change, and poor <a href="https://github.com/b4n" class="user-mention">@b4n</a> really can't be expected to review it.  There has got to be a better way, what about doing the whitespace first then?  That will have no code changes and would be more reasonably reviewable and could even be a few files at a time.</p>

<blockquote>
<p>my guts feeling is our c.c is better than the uctags one</p>
</blockquote>

<p>Thats extremely disappointing since I think so much of ours that I turn off all uses of it but the symbols pane <g-emoji alias="smile" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png">😄</g-emoji></p>

<p>(and the new uctags C/C++ one didn't seem to be much better when I tested, but presumably it will improve)</p>

<blockquote>
<p>but <a href="https://github.com/elextr" class="user-mention">@elextr</a>, it sounds like a challenge for you :-)</p>
</blockquote>

<p>I gave up self flagellation decades ago <g-emoji alias="smile" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f604.png">😄</g-emoji></p>

<p>And given your gut feeling above, no point anyway <g-emoji alias="frowning" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f626.png">😦</g-emoji> </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/1160#issuecomment-237504880">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ7iaNNWT_lOif3UPijFe_KoH2y99ks5qcbP1gaJpZM4JbITK">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJyhCEihg5-oUhog3CgANquFvMSA1ks5qcbP1gaJpZM4JbITK.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/1160#issuecomment-237504880"></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://assets-cdn.github.com/images/modules/aws/aws-bg.jpg","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 #1160: \u003e We could have a patch file similar to Scintilla which introduces the changes we need in ctags\r\n\r\nTotally, and I support the end target of your changes, just not the process of getting there :smile:\r\n\r\nI accidentally selected the \"files changed\" tab and it locked the browser for a time and used 1GB of memory  +5328 line -5619 lines.  IMNSHO its not a responsible thing to propose or to accept such a change.  I greatly admire the quality of your submissions, but even you can't be sure whats in such a change, and poor @b4n really can't be expected to review it.  There has got to be a better way, what about doing the whitespace first then?  That will have no code changes and would be more reasonably reviewable and could even be a few files at a time.\r\n\r\n\u003e my guts feeling is our c.c is better than the uctags one\r\n\r\nThats extremely disappointing since I think so much of ours that I turn off all uses of it but the symbols pane :smile:\r\n\r\n(and the new uctags C/C++ one didn't seem to be much better when I tested, but presumably it will improve)\r\n\r\n\u003e but @elextr, it sounds like a challenge for you :-)\r\n\r\nI gave up self flagellation decades ago :smile:\r\n\r\nAnd given your gut feeling above, no point anyway :frowning: "}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1160#issuecomment-237504880"}}}</script>