From 22c1d79eaeb655584c48b6cdb452a1015ca656a3 Mon Sep 17 00:00:00 2001
From: David Straub <straub@protonmail.com>
Date: Sat, 7 Nov 2020 11:00:52 +0100
Subject: [PATCH] Wrap inspect.stack in if __debug__

---
 gramps/gen/db/dummydb.py | 19 ++++++++++---------
 gramps/gen/db/utils.py   | 18 +++++++++---------
 gramps/gen/dbstate.py    | 17 +++++++++--------
 3 files changed, 28 insertions(+), 26 deletions(-)

diff --git a/gramps/gen/db/dummydb.py b/gramps/gen/db/dummydb.py
index 79b702c58..1a556d2cc 100644
--- a/gramps/gen/db/dummydb.py
+++ b/gramps/gen/db/dummydb.py
@@ -157,15 +157,16 @@ def wrapper(method):
         This 'wrapped' method logs the original function that was called, and
         where it was called from.
         """
-        class_name = args[0].__class__.__name__
-        func_name = method.__name__
-        frame = inspect.currentframe()
-        c_frame = frame.f_back
-        c_code = c_frame.f_code
-        LOG.debug('calling %s.%s()... from file %s, line %s in %s',
-                  class_name, func_name, c_code.co_filename, c_frame.f_lineno,
-                  c_code.co_name)
-        return method(*args, **keywargs)
+        if __debug__:
+            class_name = args[0].__class__.__name__
+            func_name = method.__name__
+            frame = inspect.currentframe()
+            c_frame = frame.f_back
+            c_code = c_frame.f_code
+            LOG.debug('calling %s.%s()... from file %s, line %s in %s',
+                      class_name, func_name, c_code.co_filename,
+                      c_frame.f_lineno, c_code.co_name)
+            return method(*args, **keywargs)
     return wrapped
 
 
diff --git a/gramps/gen/db/utils.py b/gramps/gen/db/utils.py
index ad9c87dc1..042425b68 100644
--- a/gramps/gen/db/utils.py
+++ b/gramps/gen/db/utils.py
@@ -71,15 +71,15 @@ def make_database(plugin_id):
         if mod:
             database = getattr(mod, pdata.databaseclass)
             db = database()
-            import inspect
-            frame = inspect.currentframe()
-            c_frame = frame.f_back
-            c_code = c_frame.f_code
-            _LOG.debug("Database class instance created Class:%s instance:%s. "
-                       "Called from File %s, line %s, in %s",
-                       db.__class__.__name__, hex(id(db)), c_code.co_filename,
-                       c_frame.f_lineno, c_code.co_name)
-
+            if __debug__:
+                import inspect
+                frame = inspect.currentframe()
+                c_frame = frame.f_back
+                c_code = c_frame.f_code
+                _LOG.debug("Database class instance created Class:%s instance:%s. "
+                           "Called from File %s, line %s, in %s",
+                           db.__class__.__name__, hex(id(db)), c_code.co_filename,
+                           c_frame.f_lineno, c_code.co_name)
             return db
         else:
             raise Exception("can't load database backend: '%s'" % plugin_id)
diff --git a/gramps/gen/dbstate.py b/gramps/gen/dbstate.py
index 6e748cafe..ee46778d3 100644
--- a/gramps/gen/dbstate.py
+++ b/gramps/gen/dbstate.py
@@ -85,14 +85,15 @@ class DbState(Callback):
         This replaces tests on DbState.open, DbState.db, DbState.db.is_open()
         and DbState.db.db_is_open all of which are deprecated.
         """
-        class_name = self.__class__.__name__
-        func_name = "is_open"
-        frame = inspect.currentframe()
-        c_frame = frame.f_back
-        c_code = c_frame.f_code
-        _LOG.debug('calling %s.%s()... from file %s, line %s in %s',
-                   class_name, func_name, c_code.co_filename, c_frame.f_lineno,
-                   c_code.co_name)
+        if __debug__:
+            class_name = self.__class__.__name__
+            func_name = "is_open"
+            frame = inspect.currentframe()
+            c_frame = frame.f_back
+            c_code = c_frame.f_code
+            _LOG.debug('calling %s.%s()... from file %s, line %s in %s',
+                       class_name, func_name, c_code.co_filename,
+                       c_frame.f_lineno, c_code.co_name)
         return (self.db is not None) and self.db.is_open()
 
     def change_database(self, database):