Remove outdated is_char check in ABI lowering (#938)

I think this predates the `string` type and was used at the time to fold
`list<char>` into an ABI string or something like that. In any case
`string` is a type nowadays and this brings the check for lifting
in-line with the one for lowering.

Closes #936

Author: alexcrichton

crates/wit-parser/src/abi.rs

@@ -1457,9 +1457,7 @@ impl<'a, B: Bindgen> Generator<'a, B> {
             Type::Id(id) => match &self.resolve.types[id].kind {
                 TypeDefKind::Type(t) => self.lift(t),
                 TypeDefKind::List(element) => {
-                    if self.is_char(element)
-                        || self.bindgen.is_list_canonical(self.resolve, element)
-                    {
+                    if self.bindgen.is_list_canonical(self.resolve, element) {
                         self.emit(&ListCanonLift { element, ty: id });
                     } else {
                         self.push_block();
@@ -1983,17 +1981,6 @@ impl<'a, B: Bindgen> Generator<'a, B> {
         });
     }
 
-    fn is_char(&self, ty: &Type) -> bool {
-        match ty {
-            Type::Char => true,
-            Type::Id(id) => match &self.resolve.types[*id].kind {
-                TypeDefKind::Type(t) => self.is_char(t),
-                _ => false,
-            },
-            _ => false,
-        }
-    }
-
     fn deallocate(&mut self, ty: &Type, addr: B::Operand, offset: i32) {
         use Instruction::*;