[Github-comments] [geany/geany] Better check for a file change on disk by also monitoring the file size. (#1175)

[Stefan Oberhumer]

> I am not sure I understand your use-case, if you change mtime backward geany won't trigger, but if the file size is changed the tarball won't be consistent anyway, so there is not reason to set mtime backward, so geany will trigger.

I don't want discuss and analyse my use-case.
Fact: If file size on disk changes the file has been changed!


> I would want to know this works for all sorts of funny file systems, for GIO and non-GIO situations etc since we have always had issues with annoying unneccessary notifications and with missed notifications. We seem to have just got that calmed down a bit and I wouldn't want to stir it up again.

I only added code to get also the file size from the ( **already used** ) functions *g_stat()* and *g_file_query_info()*.
So if they work as expected this should run on all funny file systems.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1175#issuecomment-239606748
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160813/418c097b/attachment.html>