Changes in / [71e555c:4ee3b0c1]

File:

• src/Concurrency/Keywords.cc (modified) (3 diffs)

src/Concurrency/Keywords.cc

@@ -191 +191 @@
                 void postvisit(   StructDecl * decl );
 
-                std::list<DeclarationWithType*> findMutexArgs( FunctionDecl*, bool & first );
+                std::list<DeclarationWithType*> findMutexArgs( FunctionDecl* );
                 void validate( DeclarationWithType * );
                 void addDtorStatments( FunctionDecl* func, CompoundStmt *, const std::list<DeclarationWithType * > &);
@@ -441 +441 @@
         void MutexKeyword::postvisit(FunctionDecl* decl) {
 
-                bool first = false;
-                std::list<DeclarationWithType*> mutexArgs = findMutexArgs( decl, first );
+                std::list<DeclarationWithType*> mutexArgs = findMutexArgs( decl );
                 if( mutexArgs.empty() ) return;
 
-                if( CodeGen::isConstructor(decl->name) && first ) SemanticError( decl, "constructors cannot have mutex parameters" );
+                if( CodeGen::isConstructor(decl->name) ) SemanticError( decl, "constructors cannot have mutex parameters" );
 
                 bool isDtor = CodeGen::isDestructor( decl->name );
@@ -485 +484 @@
         }
 
-        std::list<DeclarationWithType*> MutexKeyword::findMutexArgs( FunctionDecl* decl, bool & first ) {
+        std::list<DeclarationWithType*> MutexKeyword::findMutexArgs( FunctionDecl* decl ) {
                 std::list<DeclarationWithType*> mutexArgs;
 
-                bool once = true;
                 for( auto arg : decl->get_functionType()->get_parameters()) {
                         //Find mutex arguments
                         Type* ty = arg->get_type();
                         if( ! ty->get_mutex() ) continue;
-
-                        if(once) {first = true;}
-                        once = false;
 
                         //Append it to the list