zturner added inline comments. ================ Comment at: packages/Python/lldbsuite/test/lldbtest.py:1994 @@ +1993,3 @@ +# failure and retry. +def remove_file(file, num_retries = 1, sleep_duration = 0.5): + while True: ---------------- I think the default for `num_retries` should be 0. Usually you just want to try once without retry.
================ Comment at: packages/Python/lldbsuite/test/lldbtest.py:1995 @@ +1994,3 @@ +def remove_file(file, num_retries = 1, sleep_duration = 0.5): + while True: + try: ---------------- You could do `for i in range(num_retries+1):` and get rid of a bunch of the counter updating and junk http://reviews.llvm.org/D18912 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
