reapi: remove old interface for cli match allocate

Problem: the c++ bindings for the cli included two overloaded
functions for match allocate. One of those functions used a
a boolean flag as a parameter and converted it to enum to call the
newer interface. This function is no longer needed.

remove it.

Author: zekemorton

resource/reapi/bindings/c++/reapi_cli.hpp

@@ -130,10 +130,6 @@ class resource_query_t {
 
 class reapi_cli_t : public reapi_t {
 public:  
-    static int match_allocate (void *h, bool orelse_reserve,
-                               const std::string &jobspec,
-                               const uint64_t jobid, bool &reserved,
-                               std::string &R, int64_t &at, double &ov);
     static int match_allocate (void *h, match_op_t match_op,
                                const std::string &jobspec,
                                const uint64_t jobid, bool &reserved,

resource/reapi/bindings/c++/reapi_cli_impl.hpp

@@ -163,19 +163,6 @@ int reapi_cli_t::match_allocate (void *h, match_op_t match_op,
     return rc;
 }
 
-int reapi_cli_t::match_allocate (void *h, bool orelse_reserve,
-                                 const std::string &jobspec,
-                                 const uint64_t jobid, bool &reserved,
-                                 std::string &R, int64_t &at, double &ov)
-{
-    match_op_t match_op = orelse_reserve ? 
-                          match_op_t::MATCH_ALLOCATE_ORELSE_RESERVE : 
-                          match_op_t::MATCH_ALLOCATE;
-    
-    return match_allocate (h, match_op, jobspec, jobid, reserved,
-                           R, at, ov);
-}
-
 int reapi_cli_t::update_allocate (void *h, const uint64_t jobid,
                                   const std::string &R, int64_t &at, double &ov,
                                   std::string &R_out)