From 2df36164c6dfaa4c1dfbd7bb652ff9837c8d912c Mon Sep 17 00:00:00 2001
From: Joel Klinghed <the_jk@opera.com>
Date: Fri, 17 Jul 2020 12:10:54 +0200
Subject: Move IceccMonMon::connect to MonMon

All implementations are going to need a "connect" method
---
 src/icecc.cc | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

(limited to 'src/icecc.cc')

diff --git a/src/icecc.cc b/src/icecc.cc
index 5e99981..7982f7c 100644
--- a/src/icecc.cc
+++ b/src/icecc.cc
@@ -10,18 +10,19 @@
 #include "fake_monitor.hh"
 #include "icecc.hh"
 #include "monitor.hh"
+#include "monmon.hh"
 #include "pango.hh"
 
 namespace {
 
-class IceccMonMonImpl : public IceccMonMon, virtual Monitor::Observer {
+class IceccMonMon : public MonMon, virtual Monitor::Observer {
 public:
-  IceccMonMonImpl(std::shared_ptr<PollLooper> const& looper, unsigned columns)
-    : IceccMonMon(looper), connected_(false), max_jobs_(0), jobs_(0),
-      requests_(0), force_columns_(columns) {
+  IceccMonMon(std::shared_ptr<PollLooper> const& looper, unsigned columns)
+    : MonMon(looper), connected_(false), max_jobs_(0), jobs_(0), requests_(0),
+      force_columns_(columns) {
   }
 
-  void connect(Args const* args) override {
+  bool connect(Args const* args) override {
     std::unique_ptr<Monitor> monitor;
 #if FAKE_MONITOR
     monitor = FakeMonitor::create(looper_);
@@ -32,6 +33,7 @@ public:
     monitor_->add_observer(this);
     monitor_->connect(args->arg("network", ""),
                       args->arg("scheduler", ""));
+    return true;
   }
 
 protected:
@@ -348,10 +350,7 @@ private:
 
 }  // namespace
 
-IceccMonMon::IceccMonMon(std::shared_ptr<PollLooper> const& looper)
-  : MonMon(looper) {}
-
-std::unique_ptr<IceccMonMon> create_icecc_monmon(
+std::unique_ptr<MonMon> create_icecc_monmon(
     std::shared_ptr<PollLooper> const& looper, unsigned columns) {
-  return std::make_unique<IceccMonMonImpl>(looper, columns);
+  return std::make_unique<IceccMonMon>(looper, columns);
 }
-- 
cgit v1.3