From 43459fa0bb49b68cfe6238c0863f4f46682b92f9 Mon Sep 17 00:00:00 2001
From: Sosthene <contact@sosthene.net>
Date: Fri, 29 Nov 2024 08:43:43 +0100
Subject: [PATCH] Rename compute->list_modified_fields, and slight refactor

---
 src/process.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/process.rs b/src/process.rs
index 38bdafa..ff70c51 100644
--- a/src/process.rs
+++ b/src/process.rs
@@ -31,26 +31,26 @@ impl ProcessState {
         Value::Object(fields2plain)
     }
 
-    fn compute_modified_fields(&self, previous_state: Option<&ProcessState>) -> Vec<String> {
+    fn list_modified_fields(&self, previous_state: Option<&ProcessState>) -> Vec<String> {
         let new_state = &self.pcd_commitment;
 
         // Ensure the new state is a JSON object
-        let new_state_obj = new_state
+        let new_state_commitments = new_state
             .as_object()
             .expect("New state should be a JSON object");
 
         if let Some(prev_state) = previous_state {
             // Previous state exists; compute differences
-            let previous_state_obj = prev_state
+            let previous_state_commitments = prev_state
                 .pcd_commitment
                 .as_object()
                 .expect("Previous state should be a JSON object");
 
             // Compute modified fields by comparing with previous state
-            new_state_obj
+            new_state_commitments
                 .iter()
                 .filter_map(|(key, value)| {
-                    let previous_value = previous_state_obj.get(key);
+                    let previous_value = previous_state_commitments.get(key);
                     if previous_value.is_none() || value != previous_value.unwrap() {
                         Some(key.clone())
                     } else {
@@ -60,7 +60,7 @@ impl ProcessState {
                 .collect()
         } else {
             // No previous state; all fields are considered modified
-            new_state_obj.keys().cloned().collect()
+            new_state_commitments.keys().cloned().collect()
         }
     }
 
@@ -72,7 +72,7 @@ impl ProcessState {
         }
 
         // Compute modified fields
-        let modified_fields = self.compute_modified_fields(previous_state);
+        let modified_fields = self.list_modified_fields(previous_state);
 
         if modified_fields.is_empty() {
             return Err(anyhow::anyhow!("State is identical to the previous state"));