Patch to avoid re-parsing script on atime change #18

Merged
merged 1 commit into from May 11, 2011

Projects

None yet

3 participants

@jonathanperret

Hi, here's a small fix for that issue.
My scripts usually do an action (like run tests a first time) when parsed, and I was getting tired of that getting run twice when I changed the script.
I'm running watchr under Windows btw. And I love it !
-Jonathan

@rosscooperman

+1

@mynyml mynyml merged commit 726e61b into mynyml:master May 11, 2011
@mynyml
Owner
mynyml commented May 11, 2011

So I think this breaks backward compatibility because in theory you could register watches on :access events. But I'd be surprised if anyone used that.

I'm hoping to find some time to get rid of the event type feature altogether in favor of something simpler like providing prev_time and curr_time directly.

Thanks for the patch. Sorry for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment