Merge pull request #240 from KernelTuner/reduce_complexity_read_cache

improve code quality of cache file related functions

Author: benvanwerkhoven

kernel_tuner/util.py

@@ -1167,8 +1167,9 @@ def process_cache(cache, kernel_options, tuning_options, runner):
         tuning_options.cache = cached_data["cache"]
 
 
-def read_cache(cache, open_cache=True):
-    """Read the cachefile into a dictionary, if open_cache=True prepare the cachefile for appending."""
+def correct_open_cache(cache, open_cache=True):
+    """ if cache file was not properly closed, pretend it was properly closed """
+
     with open(cache, "r") as cachefile:
         filestr = cachefile.read().strip()
 
@@ -1185,6 +1186,13 @@ def read_cache(cache, open_cache=True):
             with open(cache, "w") as cachefile:
                 cachefile.write(filestr[:-3] + ",")
 
+    return filestr
+
+def read_cache(cache, open_cache=True):
+    """Read the cachefile into a dictionary, if open_cache=True prepare the cachefile for appending."""
+
+    filestr = correct_open_cache(cache, open_cache)
+
     error_configs = {
         "InvalidConfig": InvalidConfig(),
         "CompilationFailedConfig": CompilationFailedConfig(),