Changes in src/Concurrency/Waitfor.cc [08da53d:d67cdb7]

File:

• src/Concurrency/Waitfor.cc (modified) (8 diffs)

src/Concurrency/Waitfor.cc

@@ -190 +190 @@
 
                 Statement * makeAccStatement( DeclarationWithType * object, unsigned long index, const std::string & member, Expression * value, const SymTab::Indexer & indexer ) {
-                        Expression * expr = makeOpAssign(
+                        std::unique_ptr< Expression > expr( makeOpAssign(
                                 makeOpMember(
                                         makeOpIndex(
@@ -199 +199 @@
                                 ),
                                 value
-                        );
-
-                        ResolvExpr::findVoidExpression( expr, indexer );
-
-                        return new ExprStmt( noLabels, expr );
+                        ) );
+
+                        return new ExprStmt( noLabels, ResolvExpr::findVoidExpression( expr.get(), indexer ) );
                 }
 
@@ -315 +313 @@
                 stmt->push_back( new DeclStmt( noLabels, acceptables) );
 
-                Expression * set = new UntypedExpr(
+                UntypedExpr * set = new UntypedExpr(
                         new NameExpr( "__builtin_memset" ),
                         {
@@ -324 +322 @@
                 );
 
-                ResolvExpr::findVoidExpression( set, indexer );
-
-                stmt->push_back( new ExprStmt( noLabels, set ) );
+                Expression * resolved_set = ResolvExpr::findVoidExpression( set, indexer );
+                delete set;
+
+                stmt->push_back( new ExprStmt( noLabels, resolved_set ) );
 
                 return acceptables;
@@ -347 +346 @@
 
         Statement * GenerateWaitForPass::makeSetter( ObjectDecl * flag ) {
-                Expression * expr = new UntypedExpr(
+                Expression * untyped = new UntypedExpr(
                         new NameExpr( "?=?" ),
                         {
@@ -355 +354 @@
                 );
 
-                ResolvExpr::findVoidExpression( expr, indexer );
+                Expression * expr = ResolvExpr::findVoidExpression( untyped, indexer );
+                delete untyped;
 
                 return new ExprStmt( noLabels, expr );
@@ -379 +379 @@
                         new ListInit(
                                 map_range < std::list<Initializer*> > ( clause.target.arguments, [this](Expression * expr ){
-                                        Expression * init = new CastExpr(
+                                        Expression * untyped = new CastExpr(
                                                 new UntypedExpr(
                                                         new NameExpr( "get_monitor" ),
@@ -393 +393 @@
                                         );
 
-                                        ResolvExpr::findSingleExpression( init, indexer );
+                                        Expression * init = ResolvExpr::findSingleExpression( untyped, indexer );
+                                        delete untyped;
                                         return new SingleInit( init );
                                 })