Merge pull request #113 from bastion-rs/stats-sampler-improvements

Stat sampler improvements
bastion-rs · Dec 3, 2019 · 860fccb · 860fccb
2 parents 9d7c4d2 + 4191e42
commit 860fccb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/bastion-executor/src/distributor.rs b/bastion-executor/src/distributor.rs
@@ -34,9 +34,9 @@ impl Distributor {
                     placement::set_for_current(core);
 
                     // run initial stats generation for cores
-                    worker::stats_generator(core.id.clone(), &wrk);
+                    worker::stats_generator(core.id, &wrk);
                     // actual execution
-                    worker::main_loop(core.id.clone(), wrk);
+                    worker::main_loop(core.id, wrk);
                 })
                 .expect("cannot start the thread for running proc");
         }

diff --git a/bastion-executor/src/placement.rs b/bastion-executor/src/placement.rs
@@ -328,7 +328,6 @@ mod macos {
     pub fn get_core_ids() -> Option<Vec<CoreId>> {
         Some(
             (0..(num_cpus::get()))
-                .into_iter()
                 .map(|n| CoreId { id: n as usize })
                 .collect::<Vec<_>>(),
         )

diff --git a/lightproc/src/lightproc.rs b/lightproc/src/lightproc.rs
@@ -138,9 +138,9 @@ impl LightProc {
         S: Fn(LightProc) + Send + Sync + 'static,
     {
         let raw_proc = RawProc::allocate(stack, future, schedule);
-        let proc = LightProc { raw_proc: raw_proc };
+        let proc = LightProc { raw_proc };
         let handle = ProcHandle {
-            raw_proc: raw_proc,
+            raw_proc,
             _marker: PhantomData,
         };
         (proc, handle)