Added documentation to visitor methods

Author: LucaCappelletti94

src/ast/mod.rs

@@ -6457,7 +6457,7 @@ pub enum MinMaxValue {
     Empty,
     /// NO MINVALUE / NO MAXVALUE.
     None,
-    /// MINVALUE `<expr>` / MAXVALUE `<expr>`.
+    /// `MINVALUE <expr>` / `MAXVALUE <expr>`.
     Some(Expr),
 }
 
@@ -9154,7 +9154,7 @@ pub enum CopyLegacyOption {
     Delimiter(char),
     /// EMPTYASNULL
     EmptyAsNull,
-    /// ENCRYPTED \[ AUTO \]
+    /// `ENCRYPTED \[ AUTO \]`
     Encrypted {
         /// Whether `AUTO` was specified for encryption.
         auto: bool,

src/ast/value.rs

@@ -124,9 +124,10 @@ pub enum Value {
     #[cfg(not(feature = "bigdecimal"))]
     Number(String, bool),
     #[cfg(feature = "bigdecimal")]
-    // HINT: use `test_utils::number` to make an instance of
-    // Value::Number This might help if you your tests pass locally
-    // but fail on CI with the `--all-features` flag enabled
+    /// HINT: use `test_utils::number` to make an instance of
+    /// Value::Number This might help if you your tests pass locally
+    /// but fail on CI with the `--all-features` flag enabled
+    /// Numeric literal (uses `BigDecimal` when the `bigdecimal` feature is enabled).
     Number(BigDecimal, bool),
     /// 'string value'
     SingleQuotedString(String),

src/ast/visitor.rs

@@ -32,6 +32,10 @@ use core::ops::ControlFlow;
 /// #[cfg_attr(feature = "visitor", derive(Visit, VisitMut))]
 /// ```
 pub trait Visit {
+    /// Visit this node with the provided [`Visitor`].
+    ///
+    /// Implementations should call the appropriate visitor hooks to traverse
+    /// child nodes and return a `ControlFlow` value to allow early exit.
     fn visit<V: Visitor>(&self, visitor: &mut V) -> ControlFlow<V::Break>;
 }
 
@@ -47,6 +51,11 @@ pub trait Visit {
 /// #[cfg_attr(feature = "visitor", derive(Visit, VisitMut))]
 /// ```
 pub trait VisitMut {
+    /// Mutably visit this node with the provided [`VisitorMut`].
+    ///
+    /// Implementations should call the appropriate mutable visitor hooks to
+    /// traverse and allow in-place mutation of child nodes. Returning a
+    /// `ControlFlow` value permits early termination of the traversal.
     fn visit<V: VisitorMut>(&mut self, visitor: &mut V) -> ControlFlow<V::Break>;
 }