service template type should not be a pointer. Service was not being …

…found due to that.
cms-sw · Nov 22, 2024 · a1d896b · a1d896b
1 parent 92333e3
commit a1d896b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/HLTrigger/JSONMonitoring/plugins/HLTriggerJSONMonitoring.cc b/HLTrigger/JSONMonitoring/plugins/HLTriggerJSONMonitoring.cc
@@ -430,11 +430,8 @@ void HLTriggerJSONMonitoring::globalEndLuminosityBlockSummary(edm::LuminosityBlo
   unsigned int run = lumi.run();
 
   bool writeFiles = true;
-  if (edm::Service<evf::FastMonitoringService*>().isAvailable()) {
-    evf::FastMonitoringService* fms =
-        (evf::FastMonitoringService*)(edm::Service<evf::FastMonitoringService*>().operator->());
-    if (fms)
-      writeFiles = fms->shouldWriteFiles(ls);
+  if (edm::Service<evf::FastMonitoringService>().isAvailable()) {
+    writeFiles = edm::Service<evf::FastMonitoringService>()->shouldWriteFiles(ls);
   }
   if (not writeFiles)
     return;

diff --git a/HLTrigger/JSONMonitoring/plugins/L1TriggerJSONMonitoring.cc b/HLTrigger/JSONMonitoring/plugins/L1TriggerJSONMonitoring.cc
@@ -367,10 +367,7 @@ void L1TriggerJSONMonitoring::globalEndLuminosityBlockSummary(edm::LuminosityBlo
 
   bool writeFiles = true;
   if (edm::Service<evf::FastMonitoringService>().isAvailable()) {
-    evf::FastMonitoringService* fms =
-        (evf::FastMonitoringService*)(edm::Service<evf::FastMonitoringService>().operator->());
-    if (fms)
-      writeFiles = fms->shouldWriteFiles(ls);
+    writeFiles = edm::Service<evf::FastMonitoringService>()->shouldWriteFiles(ls);
   }
   if (not writeFiles)
     return;