Changeset 68f9c43 for src/SymTab/FixFunction.cc

Timestamp:

Mar 16, 2018, 5:15:02 PM (6 years ago)

Author:

Aaron Moss <a3moss@…>

Branches:

new-env, with_gc

Children:

8d7bef2

Parents:

6171841

git-author:

Aaron Moss <a3moss@…> (03/16/18 17:04:24)

git-committer:

Aaron Moss <a3moss@…> (03/16/18 17:15:02)

Message:

First pass at delete removal

File:

• src/SymTab/FixFunction.cc (modified) (2 diffs)

src/SymTab/FixFunction.cc

@@ -28 +28 @@
 
         DeclarationWithType * FixFunction::postmutate(FunctionDecl *functionDecl) {
-                // can't delete function type because it may contain assertions, so transfer ownership to new object
-                ObjectDecl *pointer = new ObjectDecl( functionDecl->name, functionDecl->get_storageClasses(), functionDecl->linkage, nullptr, new PointerType( Type::Qualifiers(), functionDecl->type ), nullptr, functionDecl->attributes );
-                functionDecl->attributes.clear();
-                functionDecl->type = nullptr;
-                delete functionDecl;
-                return pointer;
+                return new ObjectDecl{ 
+                        functionDecl->name, functionDecl->get_storageClasses(), functionDecl->linkage, nullptr, 
+                        new PointerType{ Type::Qualifiers(), functionDecl->type }, 
+                        nullptr, functionDecl->attributes };
         }
 
@@ -42 +40 @@
         Type * FixFunction::postmutate(ArrayType *arrayType) {
                 // need to recursively mutate the base type in order for multi-dimensional arrays to work.
-                PointerType *pointerType = new PointerType( arrayType->get_qualifiers(), arrayType->base, arrayType->dimension, arrayType->isVarLen, arrayType->isStatic );
-                arrayType->base = nullptr;
-                arrayType->dimension = nullptr;
-                delete arrayType;
-                return pointerType;
+                return new PointerType{ arrayType->get_qualifiers(), arrayType->base, arrayType->dimension, arrayType->isVarLen, arrayType->isStatic };
         }