Inherit constructor of satcheck_glucose_baset

This avoids having definitions of the constructors of
`satcheck_glucose_simplifiert` and `satcheck_glucose_no_simplifiert`
which were implemented the same way, by having a single template
constructor instead. This will simplify subsequent changes to the
constructor, because they will only need to be made in one place.

Author: thomasspriggs

src/solvers/sat/satcheck_glucose.cpp

@@ -243,9 +243,8 @@ void satcheck_glucose_baset<T>::set_assignment(literalt a, bool value)
 
 template <typename T>
 satcheck_glucose_baset<T>::satcheck_glucose_baset(
-  T *_solver,
   message_handlert &message_handler)
-  : cnf_solvert(message_handler), solver(_solver)
+  : cnf_solvert(message_handler), solver(new T)
 {
 }
 
@@ -285,21 +284,8 @@ void satcheck_glucose_baset<T>::set_assumptions(const bvt &bv)
   }
 }
 
-satcheck_glucose_no_simplifiert::satcheck_glucose_no_simplifiert(
-  message_handlert &message_handler)
-  : satcheck_glucose_baset<Glucose::Solver>(
-      new Glucose::Solver,
-      message_handler)
-{
-}
-
-satcheck_glucose_simplifiert::satcheck_glucose_simplifiert(
-  message_handlert &message_handler)
-  : satcheck_glucose_baset<Glucose::SimpSolver>(
-      new Glucose::SimpSolver,
-      message_handler)
-{
-}
+template class satcheck_glucose_baset<Glucose::Solver>;
+template class satcheck_glucose_baset<Glucose::SimpSolver>;
 
 void satcheck_glucose_simplifiert::set_frozen(literalt a)
 {

src/solvers/sat/satcheck_glucose.h

@@ -29,7 +29,7 @@ template <typename T>
 class satcheck_glucose_baset : public cnf_solvert, public hardness_collectort
 {
 public:
-  satcheck_glucose_baset(T *, message_handlert &message_handler);
+  explicit satcheck_glucose_baset(message_handlert &message_handler);
   virtual ~satcheck_glucose_baset();
 
   tvt l_get(literalt a) const override;
@@ -82,15 +82,15 @@ class satcheck_glucose_no_simplifiert:
   public satcheck_glucose_baset<Glucose::Solver>
 {
 public:
-  explicit satcheck_glucose_no_simplifiert(message_handlert &message_handler);
+  using satcheck_glucose_baset<Glucose::Solver>::satcheck_glucose_baset;
   const std::string solver_text() override;
 };
 
 class satcheck_glucose_simplifiert:
   public satcheck_glucose_baset<Glucose::SimpSolver>
 {
 public:
-  explicit satcheck_glucose_simplifiert(message_handlert &message_handler);
+  using satcheck_glucose_baset<Glucose::SimpSolver>::satcheck_glucose_baset;
   const std::string solver_text() override;
   void set_frozen(literalt a) override;
   bool is_eliminated(literalt a) const;