Fix a few typos and inefficient dict useage

Author: Flamefire

scorep/__main__.py

@@ -11,7 +11,6 @@ def _err_exit(msg):
 
 
 def scorep_main(argv=None):
-    # print(sys.flags)
     if argv is None:
         argv = sys.argv
 
@@ -61,10 +60,9 @@ def scorep_main(argv=None):
     if len(prog_argv) == 0:
         _err_exit("Did not find a script to run")
 
-    if ("SCOREP_PYTHON_BINDINGS_INITALISED" not in os.environ) or (
-            os.environ["SCOREP_PYTHON_BINDINGS_INITALISED"] != "true"):
+    if os.environ.get("SCOREP_PYTHON_BINDINGS_INITIALISED") != "true":
         scorep.subsystem.init_environment(scorep_config, keep_files)
-        os.environ["SCOREP_PYTHON_BINDINGS_INITALISED"] = "true"
+        os.environ["SCOREP_PYTHON_BINDINGS_INITIALISED"] = "true"
         """
         python -m starts the module as skript. i.e. sys.argv will loke like:
         ['/home/gocht/Dokumente/code/scorep_python/scorep.py', '--mpi', 'mpi_test.py']
@@ -80,7 +78,7 @@ def scorep_main(argv=None):
 
         os.execve(sys.executable, new_args, os.environ)
     else:
-        scorep.subsystem.reset_pereload()
+        scorep.subsystem.reset_preload()
 
     # everything is ready
     sys.argv = prog_argv

scorep/instrumenter.py

@@ -47,7 +47,7 @@ def unregister():
     """
     Disables the python-tracing.
     Disabling the python-tracing is more efficient than disable_recording,
-    as python does not longer call the tracing module.
+    as python does no longer call the tracing module.
     However, all the other things that are traced by Score-P will still be recorded.
     Please call register() to enable tracing again.
     """
@@ -61,7 +61,7 @@ class enable():
     with enable(region_name=None):
         do stuff
     ```
-    This overides --no-instrumenter (--nopython leagacy)
+    This overides --noinstrumenter (--nopython legacy)
     If a region name is given, the region the contextmanager is active will be marked in the trace or profile
     """
 
@@ -103,7 +103,7 @@ class disable():
     with disable():
         do stuff
     ```
-    This overides --no-instrumenter (--nopython leagacy)
+    This overides --noinstrumenter (--nopython legacy)
     If a region name is given, the region the contextmanager is active will be marked in the trace or profile
     """
 

scorep/subsystem.py

@@ -18,8 +18,8 @@ def generate_subsystem_lib_name():
 
 def generate_ld_preload(scorep_config):
     """
-    This functions generate a string that needs to be passed to $LD_PRELOAD.
-    After this sting is passed, the tracing needs to be restarted with this $LD_PRELOAD in env.
+    This functions generates a string that needs to be passed to $LD_PRELOAD.
+    After this string is passed, the tracing needs to be restarted with this $LD_PRELOAD in env.
 
     @return ld_preload string which needs to be passed to LD_PRELOAD
     """
@@ -103,15 +103,12 @@ def init_environment(scorep_config=[], keep_files=False):
     """
     Set the inital needed environmet variables, to get everythin up an running.
     As a few variables interact with LD env vars, the programms needs to be restarted after this.
-    The function set the env var `SCOREP_PYTHON_BINDINGS_INITALISED` to true, once it is done with
-    initalising.
 
     @param scorep_config configuration flags for score-p
     @param keep_files whether to keep the generated files, or not.
     """
 
-    if ("LD_PRELOAD" in os.environ) and (
-            "libscorep" in os.environ["LD_PRELOAD"]):
+    if "libscorep" in os.environ.get("LD_PRELOAD", ""):
         raise RuntimeError(
             "Score-P is already loaded. This should not happen at this point")
 
@@ -132,7 +129,7 @@ def init_environment(scorep_config=[], keep_files=False):
     os.environ["LD_PRELOAD"] = preload_str
 
 
-def reset_pereload():
+def reset_preload():
     """
     resets the environment variable `LD_PRELOAD` to the value before init_environment was called.
     """
@@ -152,6 +149,5 @@ def clean_up(keep_files=True):
     if keep_files:
         return
     else:
-        if ("SCOREP_PYTHON_BINDINGS_TEMP_DIR" in os.environ) and (
-                os.environ["SCOREP_PYTHON_BINDINGS_TEMP_DIR"] != ""):
+        if os.environ.get("SCOREP_PYTHON_BINDINGS_TEMP_DIR"):
             shutil.rmtree(os.environ["SCOREP_PYTHON_BINDINGS_TEMP_DIR"])

scorep/user.py

@@ -8,7 +8,7 @@
 def region_begin(name, file_name=None, line_number=None):
     """
     Begin of an User region. If file_name or line_number is None, both will
-    bet determined automatically
+    be determined automatically
     @param name name of the user region
     @param file_name file name of the user region
     @param line_number line number of the user region
@@ -131,7 +131,7 @@ def __exit__(self, exc_type, exc_value, traceback):
 def rewind_begin(name, file_name=None, line_number=None):
     """
     Begin of an User region. If file_name or line_number is None, both will
-    bet determined automatically
+    be determined automatically
     @param name name of the user region
     @param file_name file name of the user region
     @param line_number line number of the user region
@@ -162,7 +162,7 @@ def rewind_end(name, value):
 def oa_region_begin(name, file_name=None, line_number=None):
     """
     Begin of an Online Access region. If file_name or line_number is None, both will
-    bet determined automatically
+    be determined automatically
     @param name name of the user region
     @param file_name file name of the user region
     @param line_number line number of the user region

src/methods.cpp

@@ -151,7 +151,7 @@ extern "C"
         Py_RETURN_NONE;
     }
 
-    static PyObject* get_expiriment_dir_name(PyObject* self, PyObject* args)
+    static PyObject* get_experiment_dir_name(PyObject* self, PyObject* args)
     {
 
         return PyUnicode_FromString(SCOREP_GetExperimentDirName());
@@ -169,7 +169,7 @@ extern "C"
         { "parameter_int", parameter_int, METH_VARARGS, "User parameter int." },
         { "parameter_uint", parameter_uint, METH_VARARGS, "User parameter uint." },
         { "parameter_string", parameter_string, METH_VARARGS, "User parameter string." },
-        { "get_expiriment_dir_name", get_expiriment_dir_name, METH_VARARGS,
+        { "get_experiment_dir_name", get_experiment_dir_name, METH_VARARGS,
           "Get the Score-P experiment dir." },
         { NULL, NULL, 0, NULL } /* Sentinel */
     };