Changes in src/SymTab/Validate.cc [f53836b:bd7e609]

File:

• src/SymTab/Validate.cc (modified) (4 diffs)

src/SymTab/Validate.cc

@@ -668 +668 @@
                 }
                 filter( translationUnit, isTypedef, true );
+
         }
 
@@ -675 +676 @@
                 TypedefMap::const_iterator def = typedefNames.find( typeInst->get_name() );
                 if ( def != typedefNames.end() ) {
-                        Type *ret = def->second.first->base->clone();
+                        Type *ret = def->second.first->get_base()->clone();
                         ret->get_qualifiers() |= typeInst->get_qualifiers();
                         // place instance parameters on the typedef'd type
-                        if ( ! typeInst->parameters.empty() ) {
+                        if ( ! typeInst->get_parameters().empty() ) {
                                 ReferenceToType *rtt = dynamic_cast<ReferenceToType*>(ret);
                                 if ( ! rtt ) {
-                                        throw SemanticError("Cannot apply type parameters to base type of " + typeInst->name);
+                                        throw SemanticError("cannot apply type parameters to base type of " + typeInst->get_name());
                                 }
                                 rtt->get_parameters().clear();
-                                cloneAll( typeInst->parameters, rtt->parameters );
-                                mutateAll( rtt->parameters, *visitor );  // recursively fix typedefs on parameters
+                                cloneAll( typeInst->get_parameters(), rtt->get_parameters() );
+                                mutateAll( rtt->get_parameters(), *visitor );  // recursively fix typedefs on parameters
                         } // if
                         delete typeInst;
@@ -691 +692 @@
                 } else {
                         TypeDeclMap::const_iterator base = typedeclNames.find( typeInst->get_name() );
-                        assertf( base != typedeclNames.end(), "Cannot find typedecl name %s", typeInst->name.c_str() );
+                        assertf( base != typedeclNames.end(), "Cannot find typedecl name %s", typeInst->get_name().c_str() );
                         typeInst->set_baseType( base->second );
                 } // if
                 return typeInst;
-        }
-
-        struct VarLenChecker : WithShortCircuiting {
-                void previsit( FunctionType * ) { visit_children = false; }
-                void previsit( ArrayType * at ) {
-                        isVarLen |= at->isVarLen;
-                }
-                bool isVarLen = false;
-        };
-
-        bool isVariableLength( Type * t ) {
-                PassVisitor<VarLenChecker> varLenChecker;
-                maybeAccept( t, varLenChecker );
-                return varLenChecker.pass.isVarLen;
         }
 
@@ -719 +706 @@
                         Type * t2 = typedefNames[ tyDecl->get_name() ].first->get_base();
                         if ( ! ResolvExpr::typesCompatible( t1, t2, Indexer() ) ) {
-                                throw SemanticError( "Cannot redefine typedef: " + tyDecl->name );
-                        }
-                        // cannot redefine VLA typedefs
-                        if ( isVariableLength( t1 ) || isVariableLength( t2 ) ) {
-                                throw SemanticError( "Cannot redefine typedef: " + tyDecl->name );
+                                throw SemanticError( "cannot redefine typedef: " + tyDecl->get_name() );
                         }
                 } else {