Changeset d9bad51 for src

Timestamp:

Apr 4, 2024, 2:37:10 PM (3 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

af746cc, b6a71bc, cb98d9d

Parents:

1cfe640

Message:

Fixed memory leak in the parser.

Location:

src/Parser

Files:

• DeclarationNode.cc (modified) (1 diff)
• DeclarationNode.h (modified) (1 diff)
• parser.yy (modified) (1 diff)

src/Parser/DeclarationNode.cc

@@ -225 +225 @@
 } // DeclarationNode::newEnumValueGeneric
 
-DeclarationNode * DeclarationNode::newEnumInLine( const string name ) {
-        DeclarationNode * newnode = newName( new std::string(name) );
+DeclarationNode * DeclarationNode::newEnumInLine( const std::string * name ) {
+        DeclarationNode * newnode = newName( name );
         newnode->enumInLine = true;
         return newnode;

src/Parser/DeclarationNode.h

@@ -30 +30 @@
         static DeclarationNode * newEnumConstant( const std::string * name, ExpressionNode * constant );
         static DeclarationNode * newEnumValueGeneric( const std::string * name, InitializerNode * init );
-        static DeclarationNode * newEnumInLine( const std::string name );
+        static DeclarationNode * newEnumInLine( const std::string * name );
         static DeclarationNode * newName( const std::string * );
         static DeclarationNode * newTypeParam( ast::TypeDecl::Kind, const std::string * );

src/Parser/parser.yy

@@ -2815 +2815 @@
                 { $$ = DeclarationNode::newEnumValueGeneric( $2, $3 ); }
         | INLINE type_name
-                { $$ = DeclarationNode::newEnumInLine( *$2->symbolic.name ); }
+                {
+                        $$ = DeclarationNode::newEnumInLine( $2->symbolic.name );
+                        $2->symbolic.name = nullptr;
+                        delete $2;
+                }
         | enumerator_list ',' visible_hide_opt identifier_or_type_name enumerator_value_opt
                 { $$ = $1->set_last( DeclarationNode::newEnumValueGeneric( $4, $5 ) ); }