[Github-comments] [geany] document: show informational doc message after first reload (#672)

[Colomban Wendling]

> We should probably also add a note in the docs warning about using this option with large files, since it will "leak" any reloaded documents for the entire document's lifetime. I can just imagine someone filing a bug about "Geany leaks hundreds of megabytes when I reload many large files after changed on disk" or stuff like that.
>
> Are we definitively sure this should be enabled by default? (I'm not :)

Well… I think it's a very nice feature; but OTOH you're right that until (unless?) Scintilla starts to store a diff in the history instead of the whole change (or we do), it'll use a largish amount of memory.  And it seems very possible most people tend to reload mostly largish files (logs, DB dumps or whatnot generated outside and getting updated), so it might have a larger impact than on other features.
However, closing the document releases the memory, so it "only" impacts while the document is in use.

I guess ideally we'd reduce the memory impact the feature has, but until then I'm not 100% certain.

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/672#issuecomment-152656013
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20151030/16390926/attachment.html>