Use maximum cores for jobs=0

Fixes #207

Author: facelessuser

docs/src/markdown/about/changelog.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.12
+
+-   **NEW**: When `jobs` is set to `0`, use maximum available cores.
+
 ## 2.11
 
 -   **NEW**: Add new command line option `--skip-dict-compile` which will skip the dictionary compiling step if the

docs/src/markdown/configuration.md

@@ -19,6 +19,8 @@ process files in a given task.
 jobs: 4
 ```
 
+If `jobs` is set to `0`, the maximum available cores will be used.
+
 /// new | New 2.10
 Parallel processing is new in 2.10.
 ///

pyspelling/__init__.py

@@ -578,7 +578,7 @@ class SpellingTask:
         "O": glob.O
     }
 
-    def __init__(self, checker, config, binary='', verbose=0, jobs=0, debug=False, skip_dict_compile=False):
+    def __init__(self, checker, config, binary='', verbose=0, jobs=None, debug=False, skip_dict_compile=False):
         """Initialize."""
 
         if checker == "hunspell":  # pragma: no cover
@@ -678,12 +678,14 @@ def run_task(self, task, source_patterns=None):
             source_patterns = self.task.get('sources', [])
 
         # If jobs was not specified via command line, check the config for jobs settings
-        jobs = max(1, self.config.get('jobs', 1) if self.jobs == 0 else self.jobs)
+        jobs = self.config.get('jobs', 1) if self.jobs is None else self.jobs
+        if jobs < 0:
+            jobs = 1
 
         expect_match = self.task.get('expect_match', True)
-        if jobs > 1:
+        if jobs != 1:
             # Use multi-processing to process files concurrently
-            with ProcessPoolExecutor(max_workers=jobs) as pool:
+            with ProcessPoolExecutor(max_workers=jobs if jobs else None) as pool:
                 for results in pool.map(self.multi_check, self.walk_src(source_patterns, glob_flags, glob_limit)):
                     self.found_match = True
                     yield from results
@@ -711,7 +713,7 @@ def spellcheck(
     sources=None,
     verbose=0,
     debug=False,
-    jobs=0,
+    jobs=None,
     skip_dict_compile=False
 ):
     """Spell check."""

pyspelling/__main__.py

@@ -20,7 +20,7 @@ def main():
         '--jobs', '-j',
         action='store',
         type=int,
-        default=0,
+        default=None,
         help="Specify the number of spell checker processes to run in parallel."
     )
     parser.add_argument('--config', '-c', action='store', default='', help="Spelling config.")
@@ -64,7 +64,9 @@ def run(config, **kwargs):
     verbose = kwargs.get('verbose', 0)
     sources = kwargs.get('sources', [])
     debug = kwargs.get('debug', False)
-    jobs = kwargs.get('jobs', 0)
+    jobs = kwargs.get('jobs', None)
+    if jobs is not None and jobs < 0:
+        jobs = 1
     skip_dict_compile = kwargs.get('skip_dict_compile', False)
 
     fail = False

pyspelling/__meta__.py

@@ -188,5 +188,5 @@ def parse_version(ver):
     return Version(major, minor, micro, release, pre, post, dev)
 
 
-__version_info__ = Version(2, 11, 0, "final")
+__version_info__ = Version(2, 12, 0, "final")
 __version__ = __version_info__._get_canonical()

tests/util.py

@@ -83,7 +83,7 @@ def mktemp(self, filename, content, encoding):
         """Make temp directory."""
 
         filename = os.path.join(self.tempdir, os.path.normpath(filename))
-        base, file = os.path.split(filename)
+        base, _ = os.path.split(filename)
         if not os.path.exists(base):
             retry = 3
             while retry: