remove unnecessary mut for par_render

coder543 · coder543 · commit fb69b756f99d · 2018-05-26T19:19:40.000-07:00
diff --git a/benches/benchmark.rs b/benches/benchmark.rs
@@ -67,7 +67,7 @@ fn bench_zapper(c: &mut Criterion) {
 
     // build up a group of 100 (similar) people
     let mut group = vec![];
-    for i in 0..10000 {
+    for i in 0..100 {
         group.push(Person {
             id: 12 + i,
             name: "Bob".to_string(),
@@ -93,7 +93,7 @@ fn bench_zapper_par(c: &mut Criterion) {
         provider: "apns".to_string(),
         provider_code: 31,
     };
-    let mut bytecode = match compile(template, &env) {
+    let bytecode = match compile(template, &env) {
         Ok(bc) => bc,
         Err(err) => {
             eprintln!("error compiling template: {}", err);
@@ -105,7 +105,7 @@ fn bench_zapper_par(c: &mut Criterion) {
 
     // build up a group of 100 (similar) people
     let mut group = vec![];
-    for i in 0..10000 {
+    for i in 0..100 {
         group.push(Person {
             id: 12 + i,
             name: "Bob".to_string(),
diff --git a/src/bytecode.rs b/src/bytecode.rs
@@ -74,7 +74,7 @@ impl<
     }
 
     /// Renders a template across multiple items in parallel using Rayon with
-    /// convenient internally-managed buffers, which requires a mutable reference to self.
+    /// convenient internally-managed buffers.
     ///
     /// NOTE: This function makes serious trade-offs to enable the _maximum_ throughput.
     /// It is far less efficient, and builds up a single buffer containing all results
@@ -84,7 +84,7 @@ impl<
     /// Only use if total throughput is the sole concern.
     #[cfg(feature = "rayon")]
     pub fn par_render<'b, RunnerItem>(
-        &mut self,
+        &self,
         runner: &[RunnerItem],
         output: &mut Write,
     ) -> Result<(), Vec<::std::io::Error>>
