refactor(allocator, ast, span, ast_tools): use allocator as var nam…

…e for `Allocator` (#4900)

We mostly use `allocator` as var name for an `Allocator`, but in some places used the shorter name `alloc`. Use `allocator` everywhere for consistency.

crates/oxc_allocator/src/clone_in.rs

@@ -10,8 +10,8 @@ use crate::{Allocator, Box, Vec};
 /// ```
 /// impl<'old_alloc, 'new_alloc> CloneIn<'new_alloc> for Struct<'old_alloc> {
 ///     type Cloned = Struct<'new_alloc>;
-///     fn clone_in(&self, alloc: &'new_alloc Allocator) -> Self::Cloned {
-///         Struct { a: self.a.clone_in(alloc), b: self.b.clone_in(alloc) }
+///     fn clone_in(&self, allocator: &'new_alloc Allocator) -> Self::Cloned {
+///         Struct { a: self.a.clone_in(allocator), b: self.b.clone_in(allocator) }
 ///     }
 /// }
 /// ```
@@ -22,16 +22,16 @@ use crate::{Allocator, Box, Vec};
 pub trait CloneIn<'new_alloc>: Sized {
     type Cloned;
 
-    fn clone_in(&self, alloc: &'new_alloc Allocator) -> Self::Cloned;
+    fn clone_in(&self, allocator: &'new_alloc Allocator) -> Self::Cloned;
 }
 
 impl<'alloc, T, C> CloneIn<'alloc> for Option<T>
 where
     T: CloneIn<'alloc, Cloned = C>,
 {
     type Cloned = Option<C>;
-    fn clone_in(&self, alloc: &'alloc Allocator) -> Self::Cloned {
-        self.as_ref().map(|it| it.clone_in(alloc))
+    fn clone_in(&self, allocator: &'alloc Allocator) -> Self::Cloned {
+        self.as_ref().map(|it| it.clone_in(allocator))
     }
 }
 
@@ -40,8 +40,8 @@ where
     T: CloneIn<'new_alloc, Cloned = C>,
 {
     type Cloned = Box<'new_alloc, C>;
-    fn clone_in(&self, alloc: &'new_alloc Allocator) -> Self::Cloned {
-        Box::new_in(self.as_ref().clone_in(alloc), alloc)
+    fn clone_in(&self, allocator: &'new_alloc Allocator) -> Self::Cloned {
+        Box::new_in(self.as_ref().clone_in(allocator), allocator)
     }
 }
 
@@ -50,8 +50,8 @@ where
     T: CloneIn<'new_alloc, Cloned = C>,
 {
     type Cloned = Vec<'new_alloc, C>;
-    fn clone_in(&self, alloc: &'new_alloc Allocator) -> Self::Cloned {
-        Vec::from_iter_in(self.iter().map(|it| it.clone_in(alloc)), alloc)
+    fn clone_in(&self, allocator: &'new_alloc Allocator) -> Self::Cloned {
+        Vec::from_iter_in(self.iter().map(|it| it.clone_in(allocator)), allocator)
     }
 }
 
@@ -64,8 +64,8 @@ impl<'alloc, T: Copy> CloneIn<'alloc> for Cell<T> {
 
 impl<'old_alloc, 'new_alloc> CloneIn<'new_alloc> for &'old_alloc str {
     type Cloned = &'new_alloc str;
-    fn clone_in(&self, alloc: &'new_alloc Allocator) -> Self::Cloned {
-        alloc.alloc_str(self)
+    fn clone_in(&self, allocator: &'new_alloc Allocator) -> Self::Cloned {
+        allocator.alloc_str(self)
     }
 }
 

crates/oxc_allocator/src/convert.rs

@@ -6,14 +6,14 @@ use crate::{Allocator, Box};
 /// implementation containing blanket implementation for `IntoIn`, reflective implementation and a
 /// bunch of primitive conversions from Rust types to their arena equivalent.
 pub trait FromIn<'a, T>: Sized {
-    fn from_in(value: T, alloc: &'a Allocator) -> Self;
+    fn from_in(value: T, allocator: &'a Allocator) -> Self;
 }
 
 /// This trait works similarly to the standard library `Into` trait.
 /// It is similar to `FromIn` is reflective, A `FromIn` implementation also implicitly implements
 /// `IntoIn` for the opposite type.
 pub trait IntoIn<'a, T>: Sized {
-    fn into_in(self, alloc: &'a Allocator) -> T;
+    fn into_in(self, allocator: &'a Allocator) -> T;
 }
 
 /// `FromIn` is reflective
@@ -30,37 +30,37 @@ where
     U: FromIn<'a, T>,
 {
     #[inline]
-    fn into_in(self, alloc: &'a Allocator) -> U {
-        U::from_in(self, alloc)
+    fn into_in(self, allocator: &'a Allocator) -> U {
+        U::from_in(self, allocator)
     }
 }
 
 // ---------------- Primitive allocations ----------------
 
 impl<'a> FromIn<'a, String> for crate::String<'a> {
     #[inline(always)]
-    fn from_in(value: String, alloc: &'a Allocator) -> Self {
-        crate::String::from_str_in(value.as_str(), alloc)
+    fn from_in(value: String, allocator: &'a Allocator) -> Self {
+        crate::String::from_str_in(value.as_str(), allocator)
     }
 }
 
 impl<'a> FromIn<'a, String> for &'a str {
     #[inline(always)]
-    fn from_in(value: String, alloc: &'a Allocator) -> Self {
-        crate::String::from_str_in(value.as_str(), alloc).into_bump_str()
+    fn from_in(value: String, allocator: &'a Allocator) -> Self {
+        crate::String::from_str_in(value.as_str(), allocator).into_bump_str()
     }
 }
 
 impl<'a, T> FromIn<'a, T> for Box<'a, T> {
     #[inline(always)]
-    fn from_in(value: T, alloc: &'a Allocator) -> Self {
-        Box::new_in(value, alloc)
+    fn from_in(value: T, allocator: &'a Allocator) -> Self {
+        Box::new_in(value, allocator)
     }
 }
 
 impl<'a, T> FromIn<'a, Option<T>> for Option<Box<'a, T>> {
     #[inline(always)]
-    fn from_in(value: Option<T>, alloc: &'a Allocator) -> Self {
-        value.map(|it| Box::new_in(it, alloc))
+    fn from_in(value: Option<T>, allocator: &'a Allocator) -> Self {
+        value.map(|it| Box::new_in(it, allocator))
     }
 }

crates/oxc_ast/src/ast_impl/js.rs

@@ -702,10 +702,10 @@ impl<'a> Statement<'a> {
 }
 
 impl<'a> FromIn<'a, Expression<'a>> for Statement<'a> {
-    fn from_in(expression: Expression<'a>, alloc: &'a oxc_allocator::Allocator) -> Self {
+    fn from_in(expression: Expression<'a>, allocator: &'a oxc_allocator::Allocator) -> Self {
         Statement::ExpressionStatement(Box::from_in(
             ExpressionStatement { span: expression.span(), expression },
-            alloc,
+            allocator,
         ))
     }
 }