fix: add const keywords to be more careful to be more careful with Job * manipulations

src/broker/user_model.cpp

@@ -142,9 +142,9 @@ RoutineGreedyUser::~RoutineGreedyUser()
 {
 }
 
-bool RoutineGreedyUser::all_jobs_finished()
+bool RoutineGreedyUser::all_jobs_finished() const
 {
-    for (Job *job : last_jobs_submitted)
+    for (const Job *job : last_jobs_submitted)
     {
         if (job->status == WAITING || job->status == RUNNING
             || job->status == KILLED)

src/broker/user_model.hpp

@@ -25,7 +25,7 @@ private:
     double delay_sup;
     double delay_between_sumbit;
     int job_id = 0;
-    Job *last_job_submitted = nullptr;
+    const Job *last_job_submitted = nullptr;
 };
 
 /**
@@ -55,8 +55,8 @@ private:
     bool first_submit = true;
 
     int job_id = 0;
-    std::list<Job *> last_jobs_submitted;
-    bool all_jobs_finished();
+    std::list<const Job *> last_jobs_submitted;
+    bool all_jobs_finished() const;
 };
 
 /**