one more optimization attempt

Author: lkdvos

src/interface/interface.jl

@@ -43,16 +43,16 @@ The state is modified in-place.
 All keyword arguments are passed to the [`initialize_state!`](@ref)`(problem, algorithm, state)` function.
 """
 function solve!(problem::Problem, algorithm::Algorithm, state::State; kwargs...)
-    logger = ALGORITHM_LOGGER[]
+    logger = algorithm_logger()
     initialize_state!(problem, algorithm, state; kwargs...)
-    LOGGING_ENABLED[] && handle_message(logger, problem, algorithm, state, :Start)
+    isnothing(logger) || handle_message(logger, problem, algorithm, state, :Start)
     while !is_finished!(problem, algorithm, state)
-        LOGGING_ENABLED[] && handle_message(logger, problem, algorithm, state, :PreStep)
+        isnothing(logger) || handle_message(logger, problem, algorithm, state, :PreStep)
         increment!(state)
         step!(problem, algorithm, state)
-        LOGGING_ENABLED[] && handle_message(logger, problem, algorithm, state, :PostStep)
+        isnothing(logger) || handle_message(logger, problem, algorithm, state, :PostStep)
     end
-    LOGGING_ENABLED[] && handle_message(logger, problem, algorithm, state, :Stop)
+    isnothing(logger) || handle_message(logger, problem, algorithm, state, :Stop)
     return state
 end
 

src/logging.jl

@@ -111,6 +111,13 @@ Scoped value for handling the logging events of arbitrary algorithms.
 """
 const ALGORITHM_LOGGER = ScopedValue(AlgorithmLogger())
 
+function algorithm_logger()
+    LOGGING_ENABLED[] || return nothing
+    logger = ALGORITHM_LOGGER[]
+    isempty(logger.actions) && return nothing
+    return logger
+end
+
 # @inline here to enable the cheap global check
 @inline function log!(problem::Problem, algorithm::Algorithm, state::State, context::Symbol; kwargs...)
     if LOGGING_ENABLED[]