From ee524c9864087071259c84eae8291fa98dcd614c Mon Sep 17 00:00:00 2001
From: Ingo Weinhold <ingo_weinhold@gmx.de>
Date: Fri, 11 Feb 2005 04:51:08 +0000
Subject: [PATCH] We now also store the names of syscall parameters. This is
 used for strace. Well, not yet, but it would make a nice extension. :-)

git-svn-id: file:///srv/svn/repos/haiku/trunk/current@11338 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/tools/gensyscalls/gensyscallinfos.cpp | 46 ++++++++++++++++-------
 src/tools/gensyscalls/gensyscalls.h       |  1 +
 2 files changed, 33 insertions(+), 14 deletions(-)

diff --git a/src/tools/gensyscalls/gensyscallinfos.cpp b/src/tools/gensyscalls/gensyscallinfos.cpp
index b07a0ee5b4..912fa89417 100644
--- a/src/tools/gensyscalls/gensyscallinfos.cpp
+++ b/src/tools/gensyscalls/gensyscallinfos.cpp
@@ -43,6 +43,16 @@ struct Type {
 	string type;
 };
 
+// NamedType
+struct NamedType : Type {
+	NamedType(const char *type, const char *name) 
+		: Type(type), name(name) {}
+	NamedType(const string &type, const string &name)
+		: Type(type), name(name) {}
+
+	string name;
+};
+
 // Function
 class Function {
 public:
@@ -58,7 +68,7 @@ public:
 		return fName;
 	}
 
-	void AddParameter(const Type &type)
+	void AddParameter(const NamedType &type)
 	{
 		fParameters.push_back(type);
 	}
@@ -68,7 +78,7 @@ public:
 		return fParameters.size();
 	}
 
-	const Type &ParameterAt(int index) const
+	const NamedType &ParameterAt(int index) const
 	{
 		return fParameters[index];
 	}
@@ -84,9 +94,9 @@ public:
 	}
 
 protected:
-	string			fName;
-	vector<Type>	fParameters;
-	Type			fReturnType;
+	string				fName;
+	vector<NamedType>	fParameters;
+	Type				fReturnType;
 };
 
 // Syscall
@@ -344,9 +354,10 @@ private:
 			file << "static gensyscall_parameter_info " << paramInfoName
 				<< "[] = {" << endl;
 			for (int k = 0; k < paramCount; k++) {
-				file << "\t{ \"" << syscall.ParameterAt(k).type << "\", 0, "
-				<< "sizeof(" << syscall.ParameterAt(k).type << "), 0 },"
-				<< endl;
+				const NamedType &param = syscall.ParameterAt(k);
+				file << "\t{ \"" << param.type << "\", \"" << param.name
+				<< "\", 0, " << "sizeof(" << syscall.ParameterAt(k).type
+				<< "), 0 }," << endl;
 			}
 			file << "};" << endl;
 			file << endl;
@@ -465,16 +476,19 @@ private:
 				// that's OK
 				return;
 			}
-			throw ParseException("Error while parsing function "
-				"parameter.");
+			throw ParseException("Error while parsing function parameter.");
 		}
-		type.pop_back(); // last component is the parameter name
+
+		// last component is the parameter name
+		string typeName = type.back();
+		type.pop_back();
+
 		string typeString(type[0]);
 		for (int i = 1; i < (int)type.size(); i++) {
 			typeString += " ";
 			typeString += type[i];
 		}
-		syscall.AddParameter(typeString);
+		syscall.AddParameter(NamedType(typeString, typeName));
 	}
 
 	void _ParseFunctionPointerParameter(Tokenizer &tokenizer, Syscall &syscall,
@@ -491,9 +505,13 @@ private:
 		while (tokenizer.CheckNextToken("*"))
 			type.push_back("*");
 		// now comes the parameter name, if specified -- skip it
+		string typeName;
 		if (!tokenizer.CheckNextToken(")")) {
-			tokenizer.GetNextToken();
+			typeName = tokenizer.GetNextToken();
 			tokenizer.ExpectNextToken(")");
+		} else {
+			throw ParseException("Error while parsing function parameter. "
+				"No parameter name.");
 		}
 		type.push_back(")");
 		// the function parameters
@@ -508,7 +526,7 @@ private:
 			typeString += " ";
 			typeString += type[i];
 		}
-		syscall.AddParameter(typeString);
+		syscall.AddParameter(NamedType(typeString, typeName));
 	}
 
 private:
diff --git a/src/tools/gensyscalls/gensyscalls.h b/src/tools/gensyscalls/gensyscalls.h
index 995187e5a9..8628480810 100644
--- a/src/tools/gensyscalls/gensyscalls.h
+++ b/src/tools/gensyscalls/gensyscalls.h
@@ -5,6 +5,7 @@
 
 typedef struct gensyscall_parameter_info {
 	const char	*type;
+	const char	*name;
 	int			offset;
 	int			size;
 	int			actual_size;