Simplify descriptor.rs get() to use .cloned() directly

Copilot · ashyanSpada · web-flow · commit c66c016390a3 · 2026-04-08T06:51:45.000Z
Agent-Logs-Url: https://github.com/ashyanSpada/expression_engine_rs/sessions/e76f8236-8653-4bc9-bf47-86b983ff48e3 Co-authored-by: ashyanSpada <22587148+ashyanSpada@users.noreply.github.com>
diff --git a/src/descriptor.rs b/src/descriptor.rs
@@ -56,9 +56,7 @@ impl DescriptorManager {
 
     fn get(&self, key: DescriptorKey) -> Option<Descriptor> {
         let binding = self.store.lock().unwrap();
-        let value = binding.get(&key);
-        value?;
-        Some(value.unwrap().clone())
+        binding.get(&key).cloned()
     }
 
     pub fn set_unary_descriptor(&mut self, op: String, descriptor: Arc<UnaryDescriptor>) {

Original file line number	Diff line number	Diff line change
`@@ -56,9 +56,7 @@ impl DescriptorManager {`
`56`	`56`
`57`	`57`	`fn get(&self, key: DescriptorKey) -> Option<Descriptor> {`
`58`	`58`	`let binding = self.store.lock().unwrap();`
`59`		`- let value = binding.get(&key);`
`60`		`- value?;`
`61`		`- Some(value.unwrap().clone())`
	`59`	`+ binding.get(&key).cloned()`
`62`	`60`	`}`
`63`	`61`
`64`	`62`	`pub fn set_unary_descriptor(&mut self, op: String, descriptor: Arc<UnaryDescriptor>) {`