Changeset ca9d65e for src/Parser/DeclarationNode.cc

Timestamp:

Dec 14, 2023, 9:05:55 PM (8 months ago)

Author:

Peter A. Buhr <pabuhr@…>

Branches:

master

Children:

19a2890

Parents:

21ad568

Message:

second attempt at simplifying SemanticError? messages

File:

• src/Parser/DeclarationNode.cc (modified) (3 diffs)

src/Parser/DeclarationNode.cc

@@ -10 +10 @@
 // Created On       : Sat May 16 12:34:05 2015
 // Last Modified By : Peter A. Buhr
-// Last Modified On : Sat Jun 17 14:41:48 2023
-// Update Count     : 1405
+// Last Modified On : Thu Dec 14 19:05:17 2023
+// Update Count     : 1407
 //
 
@@ -632 +632 @@
                                         dst->basictype = src->basictype;
                                 } else if ( src->basictype != DeclarationNode::NoBasicType )
-                                        SemanticError( yylloc, string( "multiple declaration types \"" ) + DeclarationNode::basicTypeNames[ dst->basictype ] +
-                                                                   "\" and \"" + DeclarationNode::basicTypeNames[ src->basictype ] + "\"." );
-
+                                        SemanticError( yylloc, "multiple declaration types \"%s\" and \"%s\".",
+                                                                   DeclarationNode::basicTypeNames[ dst->basictype ],
+                                                                   DeclarationNode::basicTypeNames[ src->basictype ] );
                                 if ( dst->complextype == DeclarationNode::NoComplexType ) {
                                         dst->complextype = src->complextype;
                                 } else if ( src->complextype != DeclarationNode::NoComplexType )
-                                        SemanticError( yylloc, string( "multiple declaration types \"" ) + DeclarationNode::complexTypeNames[ src->complextype ] +
-                                                                   "\" and \"" + DeclarationNode::complexTypeNames[ src->complextype ] + "\"." );
-
+                                        SemanticError( yylloc, "multiple declaration types \"%s\" and \"%s\".",
+                                                                   DeclarationNode::complexTypeNames[ src->complextype ],
+                                                                   DeclarationNode::complexTypeNames[ src->complextype ] );
                                 if ( dst->signedness == DeclarationNode::NoSignedness ) {
                                         dst->signedness = src->signedness;
                                 } else if ( src->signedness != DeclarationNode::NoSignedness )
-                                        SemanticError( yylloc, string( "conflicting type specifier \"" ) + DeclarationNode::signednessNames[ dst->signedness ] +
-                                                                   "\" and \"" + DeclarationNode::signednessNames[ src->signedness ] + "\"." );
-
+                                        SemanticError( yylloc, "conflicting type specifier \"%s\" and \"%s\".",
+                                                                   DeclarationNode::signednessNames[ dst->signedness ],
+                                                                   DeclarationNode::signednessNames[ src->signedness ] );
                                 if ( dst->length == DeclarationNode::NoLength ) {
                                         dst->length = src->length;
@@ -652 +652 @@
                                         dst->length = DeclarationNode::LongLong;
                                 } else if ( src->length != DeclarationNode::NoLength )
-                                        SemanticError( yylloc, string( "conflicting type specifier \"" ) + DeclarationNode::lengthNames[ dst->length ] +
-                                                                   "\" and \"" + DeclarationNode::lengthNames[ src->length ] + "\"." );
+                                        SemanticError( yylloc, "conflicting type specifier \"%s\" and \"%s\".",
+                                                                   DeclarationNode::lengthNames[ dst->length ],
+                                                                   DeclarationNode::lengthNames[ src->length ] );
                         } // if
                         break;