Respond to review feedback

crates/wast/src/ast/types.rs

@@ -46,8 +46,8 @@ impl<'a> Parse<'a> for ValType<'a> {
         } else if l.peek::<kw::anyfunc>() {
             parser.parse::<kw::anyfunc>()?;
             Ok(ValType::Ref(RefType::Func))
-        } else if l.peek::<kw::r#externref>() {
-            parser.parse::<kw::r#externref>()?;
+        } else if l.peek::<kw::externref>() {
+            parser.parse::<kw::externref>()?;
             Ok(ValType::Ref(RefType::Extern))
         } else if l.peek::<kw::anyref>() {
             // Parse `anyref` as an alias of `externref` until all tests are
@@ -89,12 +89,21 @@ impl<'a> Parse<'a> for ValType<'a> {
 #[allow(missing_docs)]
 #[derive(Debug, PartialEq, Eq, Hash, Copy, Clone)]
 pub enum RefType<'a> {
+    /// An untyped function reference: funcref.
     Func,
+    /// A reference to any host value: externref. This was originally known as
+    /// anyref when it was the supertype of all reference value types.
     Extern,
+    /// A reference to an exception: exnref.
     Exn,
+    /// A reference that has an identity that can be compared: eqref.
     Eq,
+    /// An unboxed 31-bit integer: i31ref. This may be going away if there is no common
+    /// supertype of all reference types.
     I31,
+    /// A reference to a function, struct, or array: ref T.
     Type(ast::Index<'a>),
+    /// A nullable reference to a function, struct, or array: optref T.
     OptType(ast::Index<'a>),
 }
 
@@ -117,9 +126,6 @@ impl<'a> Parse<'a> for RefType<'a> {
         } else if l.peek::<kw::r#extern>() {
             parser.parse::<kw::r#extern>()?;
             Ok(RefType::Extern)
-        } else if l.peek::<kw::any>() {
-            parser.parse::<kw::any>()?;
-            Ok(RefType::Extern)
         } else if l.peek::<kw::exn>() {
             parser.parse::<kw::exn>()?;
             Ok(RefType::Exn)