Skip to content

Commit 843d8c7

Browse files
committed
optimize failsafe to check for None only once
1 parent 56139a3 commit 843d8c7

File tree

1 file changed

+7
-5
lines changed
  • instrumentation/opentelemetry-instrumentation-asgi/src/opentelemetry/instrumentation/asgi

1 file changed

+7
-5
lines changed

instrumentation/opentelemetry-instrumentation-asgi/src/opentelemetry/instrumentation/asgi/__init__.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -648,13 +648,15 @@ def __init__(
648648
)
649649

650650
def failsafe(func):
651+
if func is None:
652+
return None
653+
651654
@wraps(func)
652655
def wrapper(span: Span, *args, **kwargs):
653-
if func is not None:
654-
try:
655-
func(span, *args, **kwargs)
656-
except Exception as exc: # pylint: disable=broad-exception-caught
657-
span.record_exception(exc)
656+
try:
657+
func(span, *args, **kwargs)
658+
except Exception as exc: # pylint: disable=broad-exception-caught
659+
span.record_exception(exc)
658660

659661
return wrapper
660662

0 commit comments

Comments
 (0)