Fix empty range in terminate_pool_with_backoff making it a no-op

range(number_of_trials, 1) with number_of_trials=3 produces an empty
range since the default step is +1 and 3 > 1. The retry loop body
never executed, so on Windows pool termination with backoff never
happened. Fix by using range(number_of_trials) to iterate correctly.

Signed-off-by: codewithfourtix <codewithfourtix@gmail.com>

Author: codewithfourtix

src/scancode/cli.py

@@ -1390,7 +1390,7 @@ def terminate_pool(pool):
 
 def terminate_pool_with_backoff(pool, number_of_trials=3):
     # try a few times to terminate,
-    for trial in range(number_of_trials, 1):
+    for trial in range(number_of_trials):
         try:
             pool.terminate()
             break