Add braces for single-line statements

Co-authored-by: Andreas Gocht <andreas.gocht@tu-dresden.de>

Author: Flamefire

src/classes.cpp

@@ -17,10 +17,14 @@ extern "C"
     {
         scorepy::PyRefObject empty_tuple(PyTuple_New(0), scorepy::adopt_object);
         if (!empty_tuple)
+        {
             return nullptr;
+        }
         scorepy::PyRefObject empty_dict(PyDict_New(), scorepy::adopt_object);
         if (!empty_dict)
+        {
             return nullptr;
+        }
         return PyBaseObject_Type.tp_new(type, empty_tuple, empty_dict);
     }
 
@@ -37,14 +41,20 @@ extern "C"
 
         if (!PyArg_ParseTupleAndKeywords(args, kwds, "s", const_cast<char**>(kwlist),
                                          &interface_cstring))
+        {
             return -1;
+        }
 
         const std::string interface_string = interface_cstring;
         scorepy::InstrumenterInterface interface;
         if (interface_string == "Trace")
+        {
             interface = scorepy::InstrumenterInterface::Trace;
+        }
         else if (interface_string == "Profile")
+        {
             interface = scorepy::InstrumenterInterface::Profile;
+        }
         else
         {
             PyErr_Format(PyExc_TypeError, "Expected 'Trace' or 'Profile', got '%s'",
@@ -86,7 +96,9 @@ extern "C"
 
         if (!PyArg_ParseTupleAndKeywords(args, kwds, "O!sO", const_cast<char**>(kwlist),
                                          &PyFrame_Type, &frame, &event, &arg))
+        {
             return nullptr;
+        }
         return (*self)(*frame, event, arg);
     }
 }

src/scorep_bindings.cpp

@@ -22,7 +22,9 @@ PyMODINIT_FUNC PyInit__bindings(void)
 
     auto* m = PyModule_Create(&scorepmodule);
     if (!m)
+    {
         return nullptr;
+    }
 
     Py_INCREF(ctracerType);
     if (PyModule_AddObject(m, "CInstrumenter", (PyObject*)ctracerType) < 0)

src/scorepy/cInstrumenter.cpp

@@ -36,17 +36,25 @@ void CInstrumenter::enable_instrumenter()
                            adopt_object);
     }
     if (interface == InstrumenterInterface::Trace)
+    {
         PyEval_SetTrace(callback, to_PyObject());
+    }
     else
+    {
         PyEval_SetProfile(callback, to_PyObject());
+    }
 }
 
 void CInstrumenter::disable_instrumenter()
 {
     if (interface == InstrumenterInterface::Trace)
+    {
         PyEval_SetTrace(nullptr, nullptr);
+    }
     else
+    {
         PyEval_SetProfile(nullptr, nullptr);
+    }
     if (threading_set_instrumenter)
     {
         PyRefObject result(PyObject_CallFunction(threading_set_instrumenter, "O", Py_None),
@@ -65,9 +73,13 @@ int index_of(TCollection&& col, const TElement& element)
 {
     const auto it = std::find(col.cbegin(), col.cend(), element);
     if (it == col.end())
+    {
         return -1;
+    }
     else
+    {
         return std::distance(col.begin(), it);
+    }
 }
 
 // Required because:  `sys.getprofile()` returns the user object (2nd arg to PyEval_SetTrace)
@@ -81,14 +93,18 @@ PyObject* CInstrumenter::operator()(PyFrameObject& frame, const char* what_strin
     // But we might be inside a `sys.settrace` call where the user wanted to set another function
     // which would then be overwritten here. Hence use the CALL event which avoids the problem
     if (what == PyTrace_CALL)
+    {
         enable_instrumenter();
+    }
     if (on_event(frame, what, arg))
     {
         Py_INCREF(to_PyObject());
         return to_PyObject();
     }
     else
+    {
         return nullptr;
+    }
 }
 
 bool CInstrumenter::on_event(PyFrameObject& frame, int what, PyObject*)

src/scorepy/pythonHelpers.cpp

@@ -22,7 +22,9 @@ std::string get_file_name(const PyFrameObject& frame)
 {
     PyObject* filename = frame.f_code->co_filename;
     if (filename == Py_None)
+    {
         return "None";
+    }
     char actual_path[PATH_MAX];
     const char* full_file_name = realpath(PyUnicode_AsUTF8(filename), actual_path);
     return full_file_name ? full_file_name : "ErrorPath";