Rename FieldValue::as_vec() to as_vec_with(). (#263)

obi1kenobi · Apr 15, 2023 · 9868489 · 9868489
1 parent c18c6d4
commit 9868489
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/trustfall_core/src/interpreter/hints/filters.rs b/trustfall_core/src/interpreter/hints/filters.rs
@@ -49,7 +49,7 @@ pub(super) fn candidate_from_statically_evaluated_filters<'a, 'b, T: Debug + Clo
             Operation::OneOf(_, Argument::Variable(var)) => {
                 itertools::Either::Left(CandidateValue::Multiple(
                     query_variables[var.variable_name.as_ref()]
-                        .as_vec(Option::Some)
+                        .as_vec_with(Option::Some)
                         .expect("query variable was not list-typed"),
                 ))
             }

diff --git a/trustfall_core/src/ir/value.rs b/trustfall_core/src/ir/value.rs
@@ -178,7 +178,15 @@ impl FieldValue {
         }
     }
 
+    #[deprecated(since = "0.4.0", note = "renamed to `as_vec_with()`")]
     pub fn as_vec<'a, T>(&'a self, inner: impl Fn(&'a FieldValue) -> Option<T>) -> Option<Vec<T>> {
+        self.as_vec_with(inner)
+    }
+
+    pub fn as_vec_with<'a, T>(
+        &'a self,
+        inner: impl Fn(&'a FieldValue) -> Option<T>,
+    ) -> Option<Vec<T>> {
         match self {
             FieldValue::List(l) => {
                 let maybe_vec: Option<Vec<T>> = l.iter().map(inner).collect();