C++ cleanup

wrench-project · Jun 18, 2020 · 0470cdc · 0470cdc
1 parent 65a740d
commit 0470cdc
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/src/wrench/services/compute/bare_metal/BareMetalComputeService.cpp b/src/wrench/services/compute/bare_metal/BareMetalComputeService.cpp
@@ -78,7 +78,7 @@ namespace wrench {
      * @return a <cores, ram> tuple
      * @throw std::invalid_argument
      */
-    static std::tuple<std::string, unsigned long> parseResourceSpec(const std::string spec) {
+    static std::tuple<std::string, unsigned long> parseResourceSpec(const std::string &spec) {
         std::vector<std::string> tokens;
         boost::algorithm::split(tokens, spec, boost::is_any_of(":"));
         switch (tokens.size()) {

diff --git a/src/wrench/services/compute/htcondor/HTCondorComputeService.cpp b/src/wrench/services/compute/htcondor/HTCondorComputeService.cpp
@@ -339,16 +339,12 @@ namespace wrench {
         this->central_manager->submitStandardJob(job, service_specific_args);
 
         // send positive answer
-//      try {
         S4U_Mailbox::dputMessage(
                 answer_mailbox,
                 new ComputeServiceSubmitStandardJobAnswerMessage(
                         job, this->getSharedPtr<HTCondorComputeService>(), true, nullptr, this->getMessagePayloadValue(
                                 HTCondorComputeServiceMessagePayload::SUBMIT_STANDARD_JOB_ANSWER_MESSAGE_PAYLOAD)));
         return;
-//      } catch (std::shared_ptr<NetworkError> &cause) {
-//        return;
-//      }
     }
 
     /**