Removed statics from default error handler

include/spdlog/logger-inl.h

@@ -171,28 +171,20 @@ SPDLOG_INLINE bool logger::should_flush_(const details::log_msg &msg) {
 SPDLOG_INLINE void logger::err_handler_(const std::string &msg) {
     if (custom_err_handler_) {
         custom_err_handler_(msg);
-    } else {
+    }  else {
         using std::chrono::system_clock;
-        static std::mutex mutex;
-        static std::chrono::system_clock::time_point last_report_time;
-        static size_t err_counter = 0;
-        std::lock_guard<std::mutex> lk{mutex};
         auto now = system_clock::now();
-        err_counter++;
+        auto err_counter_val = err_counter.fetch_add(1, std::memory_order_relaxed)+1;
-        if (now - last_report_time < std::chrono::seconds(1)) {
-            return;
-        }
-        last_report_time = now;
         auto tm_time = details::os::localtime(system_clock::to_time_t(now));
         char date_buf[64];
         std::strftime(date_buf, sizeof(date_buf), "%Y-%m-%d %H:%M:%S", &tm_time);
-#if defined(USING_R) && defined(R_R_H)  // if in R environment
+        #if defined(USING_R) && defined(R_R_H)  // if in R environment
-        REprintf("[*** LOG ERROR #%04zu ***] [%s] [%s] %s\n", err_counter, date_buf, name().c_str(),
+            REprintf("[*** LOG ERROR #%04zu ***] [%s] [%s] %s\n",
-                 msg.c_str());
+                err_counter_val, date_buf, name().c_str(), msg.c_str());
-#else
+        #else
-        std::fprintf(stderr, "[*** LOG ERROR #%04zu ***] [%s] [%s] %s\n", err_counter, date_buf,
+            std::fprintf(stderr, "[*** LOG ERROR #%04zu ***] [%s] [%s] %s\n", err_counter_val, date_buf,
-                     name().c_str(), msg.c_str());
+                name().c_str(), msg.c_str());
-#endif
+        #endif
     }
 }
 }  // namespace spdlog

include/spdlog/logger.h

@@ -311,6 +311,7 @@ protected:
     spdlog::level_t flush_level_{level::off};
     err_handler custom_err_handler_{nullptr};
     details::backtracer tracer_;
+    std::atomic<size_t> err_counter{0};
 
     // common implementation for after templated public api has been resolved
     template <typename... Args>