<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>You don't see the whole picture. My setup to do this work looks like this:</p>

<ul>
<li>one Geany window with project containing Geany</li>
<li>one Geany window with project containing ctags</li>
<li>one Meld window with geany/ctags/parsers vs ctags/parsers diff</li>
<li>one Meld window with geany/ctags/main vs ctags/main diff</li>
</ul>

<p>and diffs are all over the place in Meld. I try to make some sense from the diffs and seeing 3700 lines less of diffs makes a huge difference for me. Also, if I didn't make the whitespace change now, every time I would copy/paste some code from uctags to Geany I would have to convert it into the old ctags indentation style if I want to have a consistent state after every commit (which I do).</p>

<p>The patches here don't make much sense by themselves, you'll only see the "why" part after you diff the result to uctags.</p>

<blockquote>
<p>One 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.</p>
</blockquote>

<p>Right, c.c is the only file file that will probably not get merged (by me at least, this is the only file except the autotools stuff I fear in Geany - but <a href="https://github.com/elextr" class="user-mention">@elextr</a>, it sounds like a challenge for you :-). The thing is it's not c.c where the main development happens in uctags - it's the completely new c/c++ parser here:</p>

<p><a href="https://github.com/universal-ctags/ctags/tree/master/parsers/cxx">https://github.com/universal-ctags/ctags/tree/master/parsers/cxx</a></p>

<p>So at some point we could switch to the new uctags parser for c/c++ and keep our c.c for the rest (my guts feeling is our c.c is better than the uctags one).</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-237473355">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ0NJWBtY1aGIiV9T407lAFloBui6ks5qcZLqgaJpZM4JbITK">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ-lJ4963IlfYRvuB8hIdq3zG8nk_ks5qcZLqgaJpZM4JbITK.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-237473355"></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":"@techee in #1160: \u003e 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.\r\n\r\nYou don't see the whole picture. My setup to do this work looks like this:\r\n\r\n- one Geany window with project containing Geany\r\n- one Geany window with project containing ctags\r\n- one Meld window with geany/ctags/parsers vs ctags/parsers diff\r\n- one Meld window with geany/ctags/main vs ctags/main diff\r\n\r\nand diffs are all over the place in Meld. I try to make some sense from the diffs and seeing 3700 lines less of diffs makes a huge difference for me. Also, if I didn't make the whitespace change now, every time I would copy/paste some code from uctags to Geany I would have to convert it into the old ctags indentation style if I want to have a consistent state after every commit (which I do).\r\n\r\nThe patches here don't make much sense by themselves, you'll only see the \"why\" part after you diff the result to uctags.\r\n\r\n\u003e One 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.\r\n\r\nRight, c.c is the only file file that will probably not get merged (by me at least, this is the only file except the autotools stuff I fear in Geany - but @elextr, it sounds like a challenge for you :-). The thing is it's not c.c where the main development happens in uctags - it's the completely new c/c++ parser here:\r\n\r\nhttps://github.com/universal-ctags/ctags/tree/master/parsers/cxx\r\n\r\nSo at some point we could switch to the new uctags parser for c/c++ and keep our c.c for the rest (my guts feeling is our c.c is better than the uctags one)."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1160#issuecomment-237473355"}}}</script>