<blockquote>
<p>The idea looks ok, though I'm not a great fan of Geany (or any program) doing magic in the background because its "likely" to be the right thing.</p>
</blockquote>

<p>Totally agree that we shouldn't do too "smart" things in Geany because these things can get really annoying. That said in this case where someone e.g. replaces all tabs with spaces I can't imagine why he would like to keep using tabs for indenting when he starts making changes in the document. I'm happy to hear a counterexample if you have one.</p>

<p>Geany already does a similar thing with line end characters - it both converts them to the new type and changes the line end type for the document.</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/1161#issuecomment-237415426">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ78DXcJrBwmfMGQo9DQMg5gVSjC9ks5qcTELgaJpZM4JcDh-">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ6EpDc3f2u66fVOkFan23BYMTcEXks5qcTELgaJpZM4JcDh-.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/1161#issuecomment-237415426"></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 #1161: \u003e The idea looks ok, though I'm not a great fan of Geany (or any program) doing magic in the background because its \"likely\" to be the right thing.\r\n\r\nTotally agree that we shouldn't do too \"smart\" things in Geany because these things can get really annoying. That said in this case where someone e.g. replaces all tabs with spaces I can't imagine why he would like to keep using tabs for indenting when he starts making changes in the document. I'm happy to hear a counterexample if you have one.\r\n\r\nGeany already does a similar thing with line end characters - it both converts them to the new type and changes the line end type for the document."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1161#issuecomment-237415426"}}}</script>