No Modify event on existing files

[SpartanJ]

Original report by Roland Rosenkranz (Bitbucket: Glatzemann, GitHub: Glatzemann).

---

System: Windows 8.1
Compiler: Visual Studio 2012

Steps to reproduce:

• use sample code from readme
• start program watching a directory containing a text file
• open the textfile and make some changes
• save the textfile

No event is triggered. When creating a new file and modifying it after creation, everything works as expected.

[SpartanJ]

Original comment by Roland Rosenkranz (Bitbucket: Glatzemann, GitHub: Glatzemann).

---

The modification is recognized when file size changed only.

[SpartanJ]

Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).

---

You are right! This is my fault, and the fix is simple, but there's a problem, IO Completion Ports will report 2 notifications events for the modified file ( it's a known issue ), and i need to add a filter to avoid reporting two times the same event. If you need a quick fix i can commit the fix without the filtering, but i prefer to commit the fix when is really fixed.

Thanks for reporting it, i'll try to fix this ASAP, since is critical.

[SpartanJ]

Original comment by Roland Rosenkranz (Bitbucket: Glatzemann, GitHub: Glatzemann).

---

I already found the simple fix myself (added FILE_NOTIFY_CHANGE_LAST_WRITE | FILE_NOTIFY_CHANGE_LAST_ACCESS to CreateWatch call in FileWatcherWin32). Now I get the two notifications you talked about. This is ok for me at this time...

I'm looking forward for a fix. Thank you very much.

BTW: Very nice project

[SpartanJ]

Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).

---

Fixed issue #14. Thanks Roland Rosenkranz for reporting it!

[SpartanJ]

Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).

---

The bug should be fixed in the last commit.
I wanted to clarify a thing: if you use FILE_NOTIFY_CHANGE_LAST_ACCESS , you will have a Modified event every time you open the file, and that's not the correct behavior. So i only used FILE_NOTIFY_CHANGE_LAST_WRITE for the fix.

Regards,
Martín

[SpartanJ]

Original comment by Roland Rosenkranz (Bitbucket: Glatzemann, GitHub: Glatzemann).

---

You're right ;-)

I tested the fix (thank you for the fast fix) and it works, but I still had to make a small source modification to get it work. I had to increase the time in WatcherWin32.cpp from 10ms to 100ms because I still got double notifications.

Maybe this should be user configureable?

[SpartanJ]

Original comment by Martín Lucas Golini (Bitbucket: SpartanJ, GitHub: SpartanJ).

---

I think that i thought a bad solution with that fix.

I've changed the way i filter this duplicate event. Now i compare the Modification Time of the modified file, if it's the same the the last reported modified event, the event is ignored. It's a little bit slower ( since i need to 'stat' the file ), but nothing problematic ( unless you are changing thousand of files per second in a slow HD ).

If you have any problem, just let me know! But i think this is the correct solution.

Regards,
Martín

[SpartanJ]

Original comment by Roland Rosenkranz (Bitbucket: Glatzemann, GitHub: Glatzemann).

---

Works like a charm. Thank you very much.