<blockquote>
<p>VCS or something changed the file on disk doesn't mean I want to lose the buffer contents</p>
</blockquote>

<p>You won't lose it, it's stored in the undo history automagically. Moreover, in this specific case of VCS, I usually would want it to auto-reload since I want to be working on the correct version of the file (though the change notification/infobar would nag me anyway sooner or later).</p>

<blockquote>
<p>Currently you will never lose the buffer contents without your explicit approval.</p>
</blockquote>

<p>Funny, it happens to me on a daily basis :)  I often accidentally press Ctrl+R or hit the Reload toolbar button instead of Save All button, then it takes me a minute to realize what happened, then I remember the change that removed the reload confirmation and then undo the reload. It's more like implicit approval :)</p>

<p>That said, this feature might interoperate poorly with the feature that stores the whole buffer change in the undo history size on reload. Imagine one is viewing a large log file that is being frequently written to, while handy to have it auto-updated in Geany, the feature storing the whole buffer in undo history would cause massive memory footprint (more than just theoretically). It might be nice to have an option to disable the storing of the buffer in undo history on reload, and possibly even make it mutually exclusive with this new option.</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/1246#issuecomment-249744623">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ76fuJk61xsOkJIOujFCEVio6nyKks5quHQsgaJpZM4KGa-x">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ3C1RwRC1osLdCYPX1MtzjiH3yykks5quHQsgaJpZM4KGa-x.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/1246#issuecomment-249744623"></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":"@codebrainz in #1246: \u003e VCS or something changed the file on disk doesn't mean I want to lose the buffer contents\r\n\r\nYou won't lose it, it's stored in the undo history automagically. Moreover, in this specific case of VCS, I usually would want it to auto-reload since I want to be working on the correct version of the file (though the change notification/infobar would nag me anyway sooner or later).\r\n\r\n\u003e Currently you will never lose the buffer contents without your explicit approval.\r\n\r\nFunny, it happens to me on a daily basis :)  I often accidentally press Ctrl+R or hit the Reload toolbar button instead of Save All button, then it takes me a minute to realize what happened, then I remember the change that removed the reload confirmation and then undo the reload. It's more like implicit approval :)\r\n\r\nThat said, this feature might interoperate poorly with the feature that stores the whole buffer change in the undo history size on reload. Imagine one is viewing a large log file that is being frequently written to, while handy to have it auto-updated in Geany, the feature storing the whole buffer in undo history would cause massive memory footprint (more than just theoretically). It might be nice to have an option to disable the storing of the buffer in undo history on reload, and possibly even make it mutually exclusive with this new option."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1246#issuecomment-249744623"}}}</script>