Make FilterResult a scoped enum.

wilx · Nov 8, 2024 · 98834f1 · 98834f1
1 parent 0e93761
commit 98834f1
Show file tree

Hide file tree

Showing 4 changed files with 87 additions and 68 deletions.
diff --git a/include/log4cplus/spi/filter.h b/include/log4cplus/spi/filter.h
@@ -49,7 +49,7 @@ namespace log4cplus {
     namespace spi {
 
 
-        enum FilterResult { DENY, /**< The log event must be dropped immediately
+        enum class FilterResult { DENY, /**< The log event must be dropped immediately
                                    *  without consulting with the remaining
                                    *  filters, if any, in the chain. */
                             NEUTRAL, /**< This filter is neutral with respect to

diff --git a/src/appender.cxx b/src/appender.cxx
@@ -364,7 +364,7 @@ Appender::syncDoAppend(const log4cplus::spi::InternalLoggingEvent& event)
 
     // Evaluate filters attached to this appender.
 
-    if (checkFilter(filter.get(), event) == spi::DENY)
+    if (checkFilter(filter.get(), event) == spi::FilterResult::DENY)
         return;
 
     // Lock system wide lock.