fix(clippy): replace map_or_else with unwrap_or_else for identity closure

tobyhede · tobyhede · commit cfffb771e112 · 2025-12-14T16:58:10.000+11:00
The identity closure |t| t in map_or_else is unnecessary.
Using unwrap_or_else is more idiomatic when the Some value
doesn't need transformation.
diff --git a/packages/cipherstash-proxy/src/postgresql/messages/bind.rs b/packages/cipherstash-proxy/src/postgresql/messages/bind.rs
@@ -104,7 +104,7 @@ fn get_param_type(idx: usize, param_types: &[i32], col: &Column) -> Type {
     param_types
         .get(idx)
         .and_then(|oid| Type::from_oid(*oid as u32))
-        .map_or_else(|| col.postgres_type.clone(), |t| t)
+        .unwrap_or_else(|| col.postgres_type.clone())
 }
 
 impl BindParam {

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ fn get_param_type(idx: usize, param_types: &[i32], col: &Column) -> Type {`
`104`	`104`	`param_types`
`105`	`105`	`.get(idx)`
`106`	`106`	`.and_then(\|oid\| Type::from_oid(*oid as u32))`
`107`		`- .map_or_else(\|\| col.postgres_type.clone(), \|t\| t)`
	`107`	`+ .unwrap_or_else(\|\| col.postgres_type.clone())`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`impl BindParam {`