minimize diff

Author: rainerschoe

src/models/feature_snapshot.rs

@@ -11,9 +11,6 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
-use std::io::Cursor;
-
-use murmur3::murmur3_32;
 
 use crate::entity::Entity;
 use crate::metering::{MeteringRecorderSender, MeteringSubject};
@@ -24,6 +21,10 @@ use crate::segment_evaluation::TargetingRules;
 
 use crate::errors::Result;
 
+use std::io::Cursor;
+
+use murmur3::murmur3_32;
+
 /// Provides a snapshot of a [`Feature`].
 #[derive(Debug)]
 pub struct FeatureSnapshot {
@@ -98,6 +99,16 @@ impl FeatureSnapshot {
         }
     }
 
+    fn calculate_normalized_hash(data: &str) -> u32 {
+        let hash = murmur3_32(&mut Cursor::new(data), 0).expect("Cannot hash the value.");
+        (f64::from(hash) / f64::from(u32::MAX) * 100.0) as u32
+    }
+
+    fn should_rollout(rollout_percentage: u32, entity: &impl Entity, feature_id: &str) -> bool {
+        let tag = format!("{}:{}", entity.get_id(), feature_id);
+        rollout_percentage == 100 || Self::calculate_normalized_hash(&tag) < rollout_percentage
+    }
+
     fn use_rollout_percentage_to_get_value_from_feature_directly(
         &self,
         entity: &impl Entity,
@@ -109,16 +120,6 @@ impl FeatureSnapshot {
             Ok(self.disabled_value.clone())
         }
     }
-
-    fn calculate_normalized_hash(data: &str) -> u32 {
-        let hash = murmur3_32(&mut Cursor::new(data), 0).expect("Cannot hash the value.");
-        (f64::from(hash) / f64::from(u32::MAX) * 100.0) as u32
-    }
-
-    fn should_rollout(rollout_percentage: u32, entity: &impl Entity, feature_id: &str) -> bool {
-        let tag = format!("{}:{}", entity.get_id(), feature_id);
-        rollout_percentage == 100 || Self::calculate_normalized_hash(&tag) < rollout_percentage
-    }
 }
 
 impl Feature for FeatureSnapshot {