Break out filesystem watcher into separate file
EventServiceManager is very large. Break out two of the functions, and
the global variables into a separate compile unit.
Code is copied as-is, with no improvements made in this patch.
Tested: At end of series.
Change-Id: I89a3605885e5bafa86a6083f1ff8c5db3bb8daf9
Signed-off-by: Ed Tanous <etanous@nvidia.com>
diff --git a/src/webserver_run.cpp b/src/webserver_run.cpp
index b037860..dfed2b7 100644
--- a/src/webserver_run.cpp
+++ b/src/webserver_run.cpp
@@ -6,6 +6,7 @@
#include "dbus_monitor.hpp"
#include "dbus_singleton.hpp"
#include "event_service_manager.hpp"
+#include "filesystem_log_watcher.hpp"
#include "google/google_service_root.hpp"
#include "hostname_monitor.hpp"
#include "ibm/management_console_rest.hpp"
@@ -117,7 +118,7 @@
if constexpr (!BMCWEB_REDFISH_DBUS_LOG)
{
- int rc = redfish::EventServiceManager::startEventLogMonitor(*io);
+ int rc = redfish::startEventLogMonitor(*io);
if (rc != 0)
{
BMCWEB_LOG_ERROR("Redfish event handler setup failed...");
@@ -142,7 +143,7 @@
crow::connections::systemBus = nullptr;
// TODO(ed) Make event log monitor an RAII object instead of global vars
- redfish::EventServiceManager::stopEventLogMonitor();
+ redfish::stopEventLogMonitor();
return 0;
}