On 27 September 2016 at 11:41, Matthew Brush <notifications@github.com><br>
wrote:<br>
<br>
> VCS or something changed the file on disk doesn't mean I want to lose the<br>
> buffer contents<br>
><br>
> You won't lose it, it's stored in the undo history automagically.<br>
> Moreover, in this specific case of VCS, I usually would want it to<br>
> auto-reload since I want to be working on the correct version of the file<br>
> (though the change notification/infobar would nag me anyway sooner or<br>
> later).<br>
><br>
<br>
​Clearly you don't accidentally checkout an old version over your changes<br>
as often as I do :(​<br>
<br>
<br>
<br>
> Currently you will never lose the buffer contents without your explicit<br>
> approval.<br>
><br>
> Funny, it happens to me on a daily basis :) I often accidentally press<br>
> Ctrl+R or hit the Reload toolbar button instead of Save All button, then it<br>
> takes me a minute to realize what happened, then I remember the change that<br>
> removed the reload confirmation and then undo the reload. It's more like<br>
> implicit approval :)<br>
><br>
<br>
​But then I almost never do this :)​<br>
<br>
<br>
<br>
> That said, this feature might interoperate poorly with the feature that<br>
> stores the whole buffer change in the undo history size on reload. Imagine<br>
> one is viewing a large log file that is being frequently written to, while<br>
> handy to have it auto-updated in Geany, the feature storing the whole<br>
> buffer in undo history would cause massive memory footprint (more than just<br>
> theoretically). It might be nice to have an option to disable the storing<br>
> of the buffer in undo history on reload, and possibly even make it mutually<br>
> exclusive with this new option.<br>
><br>
<br>
​Umm, yes, good point.  I don't think it should stop this PR since the<br>
problem exists even if you manually allow reload a few times.  But PRs are<br>
welcome to disable reload undo per document.​<br>
<br>
<br>
<br>
> —<br>
> You are receiving this because you commented.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/geany/geany/pull/1246#issuecomment-249744623>, or mute<br>
> the thread<br>
> <https://github.com/notifications/unsubscribe-auth/AAxgTbqHDsB4j1QVScLQs4nNqAK-DFt0ks5quHQsgaJpZM4KGa-x><br>
> .<br>
><br>


<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-249745451">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ2yQB89Ds9Xz3qsfqCxzW7oGxUGEks5quHXkgaJpZM4KGa-x">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ4LdPamCLtN5lK0ecG_t5AkVoYQzks5quHXkgaJpZM4KGa-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-249745451"></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":"@elextr in #1246: On 27 September 2016 at 11:41, Matthew Brush \u003cnotifications@github.com\u003e\nwrote:\n\n\u003e VCS or something changed the file on disk doesn't mean I want to lose the\n\u003e buffer contents\n\u003e\n\u003e You won't lose it, it's stored in the undo history automagically.\n\u003e Moreover, in this specific case of VCS, I usually would want it to\n\u003e auto-reload since I want to be working on the correct version of the file\n\u003e (though the change notification/infobar would nag me anyway sooner or\n\u003e later).\n\u003e\n\n​Clearly you don't accidentally checkout an old version over your changes\nas often as I do :(​\n\n\n\n\u003e Currently you will never lose the buffer contents without your explicit\n\u003e approval.\n\u003e\n\u003e Funny, it happens to me on a daily basis :) I often accidentally press\n\u003e Ctrl+R or hit the Reload toolbar button instead of Save All button, then it\n\u003e takes me a minute to realize what happened, then I remember the change that\n\u003e removed the reload confirmation and then undo the reload. It's more like\n\u003e implicit approval :)\n\u003e\n\n​But then I almost never do this :)​\n\n\n\n\u003e That said, this feature might interoperate poorly with the feature that\n\u003e stores the whole buffer change in the undo history size on reload. Imagine\n\u003e one is viewing a large log file that is being frequently written to, while\n\u003e handy to have it auto-updated in Geany, the feature storing the whole\n\u003e buffer in undo history would cause massive memory footprint (more than just\n\u003e theoretically). It might be nice to have an option to disable the storing\n\u003e of the buffer in undo history on reload, and possibly even make it mutually\n\u003e exclusive with this new option.\n\u003e\n\n​Umm, yes, good point.  I don't think it should stop this PR since the\nproblem exists even if you manually allow reload a few times.  But PRs are\nwelcome to disable reload undo per document.​\n\n\n\n\u003e —\n\u003e You are receiving this because you commented.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/geany/geany/pull/1246#issuecomment-249744623\u003e, or mute\n\u003e the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AAxgTbqHDsB4j1QVScLQs4nNqAK-DFt0ks5quHQsgaJpZM4KGa-x\u003e\n\u003e .\n\u003e\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1246#issuecomment-249745451"}}}</script>