screen: move each change along instead of copying it

Screening copied every change it took, and every one left waiting for
a signature, bytes and all, from the batch it had ordered and then
dropped. Move them out of the batch instead; only a refused change,
recorded with why, is still copied.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/access/screen.rs b/src/access/screen.rs
index 0ff7ff2..68e4db7 100644
--- a/src/access/screen.rs
+++ b/src/access/screen.rs
@@ -389,7 +389,7 @@ pub(crate) fn apply(
     // The values their users sign, under a strict policy, as the document holds them:
     // read again once the document takes the accepted (`reaches_signed`).
     let mut signed: Option<Option<Vec<Vec<String>>>> = None;
-    for (index, change) in ready.iter().enumerate() {
+    for (index, change) in ready.into_iter().enumerate() {
         let hash = change.hash();
         let committer = committers[index].as_deref();
         let user = change.message().and_then(user_of).map(str::to_owned);
@@ -420,7 +420,7 @@ pub(crate) fn apply(
         // whoever brought it, not by the node the actor is.
         if signatures
             .as_deref()
-            .is_some_and(|signatures| signatures.forks(doc, change))
+            .is_some_and(|signatures| signatures.forks(doc, &change))
         {
             refused.insert(hash, hash);
             forged.insert(hash);
@@ -442,7 +442,7 @@ pub(crate) fn apply(
             || change.deps().iter().any(|dep| unsigned.contains(dep))
         {
             unsigned.insert(hash);
-            waiting.0.push(change.clone());
+            waiting.0.push(change);
             continue;
         }
         // It makes a map where one accepted before may have: the rules read whether one
@@ -489,7 +489,7 @@ pub(crate) fn apply(
         };
         match judged {
             Ok(()) => {
-                accepted.push(change.clone());
+                accepted.push(change);
                 if let Some(committer) = committer {
                     screened.taken_from.insert(committer.to_owned());
                 }