<blockquote>
<p>There's no reason to spend much time on reviewing the whitespace stuff - all the files will eventually get replaced by the current uctags files.</p>
</blockquote>

<p>So why bother at all? Don't make extra work for yourself and others :)  Especially just for whitespace :) And if it causes conflicts with real changes, more reason to drop or delay the whitespace fixes.</p>

<blockquote>
<p>I want to make sure I don't miss some of the Geany's patches of ctags so I'm doing it gradually.</p>
</blockquote>

<p>Yes thats the right way.  </p>

<p>One concern I have with this whole process is its gonna come to a screaming halt at <code>c.c</code> and not get completed, preventing us from getting most of the benefit of upstream compatibility.  IIUC the geany version of <code>c.c</code> has several extra languages mixed into it.  Maybe that should be looked at first.</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-237416180">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ4040PzivSYc5qie83a4fXay7jhZks5qcTI0gaJpZM4JbITK">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ0nJFT1ZeJy9FDynVNkkhdQ6u6A-ks5qcTI0gaJpZM4JbITK.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-237416180"></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 There's no reason to spend much time on reviewing the whitespace stuff - all the files will eventually get replaced by the current uctags files.\r\n\r\nSo why bother at all? Don't make extra work for yourself and others :)  Especially just for whitespace :) And if it causes conflicts with real changes, more reason to drop or delay the whitespace fixes.\r\n\r\n\u003e  I want to make sure I don't miss some of the Geany's patches of ctags so I'm doing it gradually.\r\n\r\nYes thats the right way.  \r\n\r\nOne concern I have with this whole process is its gonna come to a screaming halt at `c.c` and not get completed, preventing us from getting most of the benefit of upstream compatibility.  IIUC the geany version of `c.c` has several extra languages mixed into it.  Maybe that should be looked at first."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1160#issuecomment-237416180"}}}</script>