Inline active_status

andreas-el · Aug 29, 2023 · 59b5851 · 59b5851
1 parent e3f595f
commit 59b5851
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/clib/lib/job_queue/slurm_driver.cpp b/src/clib/lib/job_queue/slurm_driver.cpp
@@ -43,10 +43,6 @@ class SlurmStatus {
 
     void new_job(int job_id) { this->update(job_id, JOB_QUEUE_PENDING); }
 
-    static bool active_status(job_status_type status) {
-        return (status == JOB_QUEUE_PENDING || status == JOB_QUEUE_RUNNING);
-    }
-
     /**
     This function is used when the status of the jobs is updated with squeue. The
     semantics is as follows:
@@ -75,7 +71,8 @@ class SlurmStatus {
 
             auto squeue_pair = squeue_jobs.find(job_id);
             if (squeue_pair == squeue_jobs.end()) {
-                if (this->active_status(job_status))
+                if ((job_status == JOB_QUEUE_PENDING ||
+                     job_status == JOB_QUEUE_RUNNING))
                     active_jobs.push_back(job_id);
             } else
                 this->jobs[job_id] = squeue_pair->second;