<blockquote>
<p>It looks to me like we can remove the auto_reload option and make it automatic<br>
without data loss.</p>
</blockquote>

<p><a href="https://github.com/shiftee" class="user-mention">@shiftee</a> two ways you can lose data with this scheme:</p>

<p>1) a modified file that has been saved (so the buffer is clean) and is then changed on disk may lose the changes that are still in the buffer if it always reloaded without asking. Think like "Oh, I'll just checkout file from Git to see what it has". Git replaces the modified file. Geany notices, reloads and BANG your changes are lost.</p>

<p>2) when the OOM killer closes geany without saving the files because it consumed too much memory :)</p>

<blockquote>
<p>I agree, but for the moment it's probably wise to keep it as is since it may cause high-memory usage.</p>
</blockquote>

<p>Agree with <a href="https://github.com/codebrainz" class="user-mention">@codebrainz</a> that the option is needed, think of a log file that is constantly updating, it will keep reloading and then save each of the previous contents in the undo buffer in memory.  For a large logfile it could multiply memory usage very quickly.</p>

<blockquote>
<p>The only potential change I can think might be to change the default enabled/disabled state of it, or perhaps making it mutually exclusive with the new auto_reload option.</p>
</blockquote>

<p>Would fix the memory usage, but then you would lose the undo protection against accidental reloads any time auto reload was on.</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-250638892">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ6jnHx1Owd4yRSjn6grNG0HKveRkks5qvGpigaJpZM4KGa-x">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJw6yd3xzcCrji57BaBfChaEEXgx_ks5qvGpigaJpZM4KGa-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-250638892"></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: \u003e It looks to me like we can remove the auto_reload option and make it automatic\r\nwithout data loss.\r\n\r\n@shiftee two ways you can lose data with this scheme:\r\n\r\n1) a modified file that has been saved (so the buffer is clean) and is then changed on disk may lose the changes that are still in the buffer if it always reloaded without asking. Think like \"Oh, I'll just checkout file from Git to see what it has\". Git replaces the modified file. Geany notices, reloads and BANG your changes are lost.\r\n\r\n2) when the OOM killer closes geany without saving the files because it consumed too much memory :)\r\n\r\n\u003e I agree, but for the moment it's probably wise to keep it as is since it may cause high-memory usage.\r\n\r\nAgree with @codebrainz that the option is needed, think of a log file that is constantly updating, it will keep reloading and then save each of the previous contents in the undo buffer in memory.  For a large logfile it could multiply memory usage very quickly.\r\n\r\n\u003e The only potential change I can think might be to change the default enabled/disabled state of it, or perhaps making it mutually exclusive with the new auto_reload option.\r\n\r\nWould fix the memory usage, but then you would lose the undo protection against accidental reloads any time auto reload was on.\r\n\r\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1246#issuecomment-250638892"}}}</script>