testpython.git

@@ -3,11 +3,11 @@

 import threading, queue

-_counter = 0

+_counter:int = 0

 counter_queue = queue.Queue()

-def counter_manager():

+def counter_manager() -> None:

     'I have EXCLUSIVE rights to update the _counter variable'

     global _counter

@@ -22,7 +22,7 @@ def counter_manager():

 print_queue = queue.Queue()

-def print_manager():

+def print_manager() -> None:

     'I have EXCLUSIVE rights to call the "print" keyword'

     while True:

         job = print_queue.get()

@@ -31,12 +31,12 @@ def print_manager():

         print_queue.task_done()

-def worker():

+def worker() -> None:

     'My job is to increment the counter and print the current count'

     counter_queue.put(1)

-def run():

+def run() -> None:

     # Set up the counter manager

     t = threading.Thread(target=counter_manager)

     t.daemon = True

@@ -4,8 +4,9 @@ import os

 import urllib.request

 import urllib.error

 from multiprocessing.pool import ThreadPool

+from typing import List, Tuple, Union

-sites = [

+sites: List[str] = [

 #    'https://www.yahoo.com/',

 #    'http://www.cnn.com',

 #    'http://www.python.org',

@@ -23,7 +24,7 @@ sites = [

 ]

-def sitesize(url: str):

+def sitesize(url: str) -> Tuple[str, Union[int, str]]:

     ''' Determine the size of a website '''

     try:

         with urllib.request.urlopen(url) as f:

@@ -32,10 +33,10 @@ def sitesize(url: str):

     except urllib.error.HTTPError as e:

         return url, str(e)

     except urllib.error.URLError as e:

-        return url, "On macOS? Try another python3. " + str(e)

+        return url, str(e)

-def run():

+def run() -> None:

     # Network bound: Use max(); CPU bound: Use min()

     pool = ThreadPool(max(len(sites), os.cpu_count()))

     for result in pool.imap_unordered(sitesize, sites):

@@ -9,6 +9,7 @@ import functools

 from argparse import ArgumentParser

 import counter.counter

 import sitesize.sitesize

+from typing import Optional, Callable

 def timeit(f):

@@ -22,7 +23,8 @@ def timeit(f):

     return wrapper

-def set_v_print(verbose: bool):

+v_print:Callable

+def set_v_print(verbose: bool) -> None:

     """

     Defines the function v_print.

     It prints if verbose is true, otherwise, it does nothing.

@@ -51,7 +53,7 @@ class Coffee:

 @timeit

-def main(debug: bool):

+def main(debug: bool) -> None:

     script_dir = os.path.abspath(os.path.dirname(sys.argv[0]))

     print(f'{sys.argv[0]} is in {script_dir}.')