Skip to content
This repository has been archived by the owner on Dec 10, 2019. It is now read-only.

missing timer.cancel() in main #43

Closed
maverickwoo opened this issue Jul 30, 2013 · 1 comment
Closed

missing timer.cancel() in main #43

maverickwoo opened this issue Jul 30, 2013 · 1 comment

Comments

@maverickwoo
Copy link

Since 3ecd930, specifying a timeout actually causes vcprompt to run for exactly that many milliseconds. Tested on Python 2.7.1 in OS X. Patch based on 6a10066:

diff --git a/bin/vcprompt b/bin/vcprompt
index e8880f1..9e1dbb6 100755
--- a/bin/vcprompt
+++ b/bin/vcprompt
@@ -235,6 +235,7 @@ def main():
     output = vcprompt(options)
     if options.newline:
         output += '\n'
+    timer.cancel()
     return output
@djl
Copy link
Owner

djl commented Jul 30, 2013

Fixed in 164220d. Thanks!

@djl djl closed this as completed Jul 30, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants