Code review comments

Author: yoavcloud

src/ast/mod.rs

@@ -8713,18 +8713,18 @@ impl fmt::Display for CopyLegacyOption {
             CompUpdate { preset, enabled } => {
                 write!(f, "COMPUPDATE")?;
                 if *preset {
-                    write!(f, " PRESET")
-                } else {
+                    write!(f, " PRESET")?;
+                } else if let Some(enabled) = enabled {
                     write!(
                         f,
                         "{}",
-                        match *enabled {
-                            Some(true) => " TRUE",
-                            Some(false) => " FALSE",
-                            _ => "",
+                        match enabled {
+                            true => " TRUE",
+                            false => " FALSE",
                         }
-                    )
+                    )?;
                 }
+                Ok(())
             }
             Csv(opts) => {
                 write!(f, "CSV")?;

src/parser/mod.rs

@@ -9684,7 +9684,6 @@ impl<'a> Parser<'a> {
                     Keyword::ON,
                     Keyword::OFF,
                 ]) {
-                    Some(Keyword::PRESET) => None,
                     Some(Keyword::TRUE) | Some(Keyword::ON) => Some(true),
                     Some(Keyword::FALSE) | Some(Keyword::OFF) => Some(false),
                     _ => None,