The function `document_check_disk_status()` must be disentangled for a [Reload 
all](https://github.com/geany/geany/issues/2540) feature as first **all** file 
modifications have to be examined before **any** correct display warning can be 
created.

Furthermore, I interpret the value of `doc->changed` as a modification of the 
text buffer in RAM, and `doc->priv->file_disk_status` as the interpretation of 
the disk on file. A modification of the file on disk should not set 
`doc->changed` to TRUE, as we may still close the application without losing 
data.
Some work in progress is visible 
[here](https://github.com/peter-scholtens/geany/tree/re-load-all). Known issues:
1. GUI is messy, but already report other files, just to get the idea.
2. GUI always has five buttons, but the amount of unmodified files or others 
may be zero.
3. Response of `RESPONSE_DOCUMENT_RELOAD_ALL` is not finished.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3711#issuecomment-1848357521
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/issues/3711/[email protected]>

Reply via email to