fix incorrect error for invalid type (#484)

In a specific situation, the parser was "losing" error messages and that
was causing big problems downstream.

pkg/corset/parser.go

@@ -193,8 +193,8 @@ func (p *Parser) parseModuleContents(module string, terms []sexp.SExp) ([]Declar
 			err := p.translator.SyntaxError(s, "unexpected or malformed declaration")
 			errors = append(errors, *err)
 		} else if e.MatchSymbols(2, "module") {
-			return decls, terms[i:], nil
-		} else if decl, errs := p.parseDeclaration(module, e); errs != nil {
+			return decls, terms[i:], errors
+		} else if decl, errs := p.parseDeclaration(module, e); len(errs) > 0 {
 			errors = append(errors, errs...)
 		} else {
 			// Continue accumulating declarations for this module.

pkg/test/invalid_corset_test.go

@@ -68,6 +68,10 @@ func Test_Invalid_Basic_12(t *testing.T) {
 	CheckInvalid(t, "basic_invalid_12")
 }
 
+func Test_Invalid_Basic_13(t *testing.T) {
+	CheckInvalid(t, "basic_invalid_13")
+}
+
 // ===================================================================
 // Constant Tests
 // ===================================================================

testdata/basic_invalid_13.lisp

@@ -0,0 +1,7 @@
+;;error:4:22-26:unknown type
+(module mmio)
+
+(defcolumns (COUNTER :dac))
+(defalias CT COUNTER)
+
+(module mmio)