removed constructor that takes a margo instance

include/alpha/Provider.hpp

@@ -36,19 +36,6 @@ class Provider {
              const std::string& config,
              const tl::pool& pool = tl::pool());
 
-    /**
-     * @brief Constructor.
-     *
-     * @param mid Margo instance id to use to receive RPCs.
-     * @param provider_id Provider id.
-     * @param config JSON-formatted configuration.
-     * @param pool Argobots pool to use to handle RPCs.
-     */
-    Provider(margo_instance_id mid,
-             uint16_t provider_id,
-             const std::string& config,
-             const tl::pool& pool = tl::pool());
-
     /**
      * @brief Copy-constructor is deleted.
      */

src/Provider.cpp

@@ -18,11 +18,6 @@ Provider::Provider(const tl::engine& engine, uint16_t provider_id, const std::st
     self->get_engine().push_finalize_callback(this, [p=this]() { p->self.reset(); });
 }
 
-Provider::Provider(margo_instance_id mid, uint16_t provider_id, const std::string& config, const tl::pool& p)
-: self(std::make_shared<ProviderImpl>(mid, provider_id, config, p)) {
-    self->get_engine().push_finalize_callback(this, [p=this]() { p->self.reset(); });
-}
-
 Provider::Provider(Provider&& other) {
     other.self->get_engine().pop_finalize_callback(this);
     self = std::move(other.self);