Changeset 44adf1b for src/Parser/TypeData.cc

Timestamp:

Mar 5, 2024, 9:55:04 AM (3 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

af60383

Parents:

9262fe9

Message:

Removed casts around get_next (also replaced with direct field access) that are no longer needed.

File:

• src/Parser/TypeData.cc (modified) (5 diffs)

src/Parser/TypeData.cc

@@ -494 +494 @@
         for ( auto i = outputList.begin() ;
                         i != outputList.end() ;
-                        ++i, n = (DeclarationNode*)n->get_next() ) {
+                        ++i, n = n->next ) {
                 // Only the object type class adds additional assertions.
                 if ( n->variable.tyClass != ast::TypeDecl::Otype ) {
@@ -639 +639 @@
         for ( auto i = outputForall.begin() ;
                         i != outputForall.end() ;
-                        ++i, n = (DeclarationNode*)n->get_next() ) {
+                        ++i, n = n->next ) {
                 // Only the object type class adds additional assertions.
                 if ( n->variable.tyClass != ast::TypeDecl::Otype ) {
@@ -1272 +1272 @@
         auto members = ret->members.begin();
         ret->hide = td->enumeration.hiding == EnumHiding::Hide ? ast::EnumDecl::EnumHiding::Hide : ast::EnumDecl::EnumHiding::Visible;
-        for ( const DeclarationNode * cur = td->enumeration.constants; cur != nullptr; cur = dynamic_cast< DeclarationNode * >( cur->get_next() ), ++members ) {
+        for ( const DeclarationNode * cur = td->enumeration.constants; cur != nullptr; cur = cur->next, ++members ) {
                 if ( cur->enumInLine ) {
                         // Do Nothing
@@ -1500 +1500 @@
         assert( ! function.params );
         // loop over declaration first as it is easier to spot errors
-        for ( DeclarationNode * decl = function.oldDeclList; decl != nullptr; decl = dynamic_cast< DeclarationNode * >( decl->get_next() ) ) {
+        for ( DeclarationNode * decl = function.oldDeclList; decl != nullptr; decl = decl->next ) {
                 // scan ALL parameter names for each declaration name to check for duplicates
-                for ( DeclarationNode * param = function.idList; param != nullptr; param = dynamic_cast< DeclarationNode * >( param->get_next() ) ) {
+                for ( DeclarationNode * param = function.idList; param != nullptr; param = param->next ) {
                         if ( *decl->name == *param->name ) {
                                 // type set => parameter name already transformed by a declaration names so there is a duplicate
@@ -1524 +1524 @@
         //    rtb( a, b, c ) const char * b; {} => int rtn( int a, const char * b, int c ) {}
 
-        for ( DeclarationNode * param = function.idList; param != nullptr; param = dynamic_cast< DeclarationNode * >( param->get_next() ) ) {
+        for ( DeclarationNode * param = function.idList; param != nullptr; param = param->next ) {
                 if ( ! param->type ) {                                                  // generate type int for empty parameter type
                         param->type = new TypeData( TypeData::Basic );