From 0a6d9f4dc993dc8d7d68204681ed83e56fa9c77a Mon Sep 17 00:00:00 2001
From: Sean Luchen <seanluchen@google.com>
Date: Mon, 31 Mar 2025 10:44:23 -0700
Subject: [PATCH] Factor report_unexpected_token out into its own function.

---
 passes/techmap/libparse.cc | 69 ++++++++++++++++----------------------
 passes/techmap/libparse.h  |  1 +
 2 files changed, 30 insertions(+), 40 deletions(-)

diff --git a/passes/techmap/libparse.cc b/passes/techmap/libparse.cc
index 06dd6288e..afc49b454 100644
--- a/passes/techmap/libparse.cc
+++ b/passes/techmap/libparse.cc
@@ -332,6 +332,32 @@ int LibertyParser::lexer(std::string &str)
 	return c;
 }
 
+void LibertyParser::report_unexpected_token(int tok)
+{
+	std::string eReport;
+	switch(tok)
+	{
+	case 'n':
+		error("Unexpected newline.");
+		break;
+	case '[':
+	case ']':
+	case '}':
+	case '{':
+	case '\"':
+	case ':':
+		eReport = "Unexpected '";
+		eReport += static_cast<char>(tok);
+		eReport += "'.";
+		error(eReport);
+		break;
+	default:
+		eReport = "Unexpected token: ";
+		eReport += static_cast<char>(tok);
+		error(eReport);
+	}
+}
+
 LibertyAst *LibertyParser::parse()
 {
 	std::string str;
@@ -350,26 +376,7 @@ LibertyAst *LibertyParser::parse()
 		return NULL;
 
 	if (tok != 'v') {
-		std::string eReport;
-		switch(tok)
-		{
-		case 'n':
-			error("Unexpected newline.");
-			break;
-		case '[':
-		case ']':
-		case '}':
-		case '{':
-		case '\"':
-		case ':':
-			eReport = "Unexpected '";
-			eReport += static_cast<char>(tok);
-			eReport += "'.";
-			error(eReport);
-			break;
-		default:
-			error();
-		}
+		report_unexpected_token(tok);	
 	}
 
 	LibertyAst *ast = new LibertyAst;
@@ -460,25 +467,7 @@ LibertyAst *LibertyParser::parse()
 					continue;           
 				}
 				if (tok != 'v') {
-					std::string eReport;
-					switch(tok)
-					{
-					case 'n':
-					  continue;
-					case '[':
-					case ']':
-					case '}':
-					case '{':
-					case '\"':
-					case ':':
-						eReport = "Unexpected '";
-						eReport += static_cast<char>(tok);
-						eReport += "'.";
-						error(eReport);
-						break;
-					default:
-						error();
-					}
+					report_unexpected_token(tok);
 				}
 				ast->args.push_back(arg);
 			}
@@ -495,7 +484,7 @@ LibertyAst *LibertyParser::parse()
 			break;
 		}
 
-		error();
+		report_unexpected_token(tok);
 	}
 
 	return ast;
diff --git a/passes/techmap/libparse.h b/passes/techmap/libparse.h
index 16808fc58..ea81abe2f 100644
--- a/passes/techmap/libparse.h
+++ b/passes/techmap/libparse.h
@@ -105,6 +105,7 @@ namespace Yosys
 		*/
 		int lexer(std::string &str);
 
+    void report_unexpected_token(int tok);
 		LibertyAst *parse();
 		void error() const;
 		void error(const std::string &str) const;