ECPG: Fix searching for quoted cursor names case-sensitively.

Michael Meskes · Michael Meskes · commit 9a1c6dd43a11 · 2013-11-27T11:15:34.000+01:00
Patch by Böszörményi Zoltán &lt;zb@cybertec.at&gt;
diff --git a/src/interfaces/ecpg/preproc/ecpg.addons b/src/interfaces/ecpg/preproc/ecpg.addons
@@ -296,7 +296,7 @@ ECPG: DeclareCursorStmtDECLAREcursor_namecursor_optionsCURSORopt_holdFORSelectSt
 		struct cursor *ptr, *this;
 		char *cursor_marker = $2[0] == ':' ? mm_strdup("$0") : mm_strdup($2);
 		char *comment, *c1, *c2;
-		int (* strcmp_fn)(const char *, const char *) = ($2[0] == ':' ? strcmp : pg_strcasecmp);
+		int (* strcmp_fn)(const char *, const char *) = (($2[0] == ':' || $2[0] == '"') ? strcmp : pg_strcasecmp);
 
 		for (ptr = cur; ptr != NULL; ptr = ptr->next)
 		{
diff --git a/src/interfaces/ecpg/preproc/ecpg.header b/src/interfaces/ecpg/preproc/ecpg.header
@@ -377,7 +377,7 @@ add_additional_variables(char *name, bool insert)
 {
 	struct cursor *ptr;
 	struct arguments *p;
-	int (* strcmp_fn)(const char *, const char *) = (name[0] == ':' ? strcmp : pg_strcasecmp);
+	int (* strcmp_fn)(const char *, const char *) = ((name[0] == ':' || name[0] == '"') ? strcmp : pg_strcasecmp);
 
 	for (ptr = cur; ptr != NULL; ptr=ptr->next)
 	{
diff --git a/src/interfaces/ecpg/preproc/ecpg.trailer b/src/interfaces/ecpg/preproc/ecpg.trailer
@@ -289,7 +289,7 @@ ECPGCursorStmt:  DECLARE cursor_name cursor_options CURSOR opt_hold FOR prepared
 		{
 			struct cursor *ptr, *this;
 			char *cursor_marker = $2[0] == ':' ? mm_strdup("$0") : mm_strdup($2);
-			int (* strcmp_fn)(const char *, const char *) = ($2[0] == ':' ? strcmp : pg_strcasecmp);
+			int (* strcmp_fn)(const char *, const char *) = (($2[0] == ':' || $2[0] == '"') ? strcmp : pg_strcasecmp);
 			struct variable *thisquery = (struct variable *)mm_alloc(sizeof(struct variable));
 			const char *con = connection ? connection : "NULL";
 			char *comment;

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ ECPG: DeclareCursorStmtDECLAREcursor_namecursor_optionsCURSORopt_holdFORSelectSt`
`296`	`296`	`struct cursor ptr, this;`
`297`	`297`	`char *cursor_marker = $2[0] == ':' ? mm_strdup("$0") : mm_strdup($2);`
`298`	`298`	`char comment, c1, *c2;`
`299`		`- int (* strcmp_fn)(const char , const char ) = ($2[0] == ':' ? strcmp : pg_strcasecmp);`
	`299`	`+ int (* strcmp_fn)(const char , const char ) = (($2[0] == ':' \|\| $2[0] == '"') ? strcmp : pg_strcasecmp);`
`300`	`300`
`301`	`301`	`for (ptr = cur; ptr != NULL; ptr = ptr->next)`
`302`	`302`	`{`
Original file line number	Diff line number	Diff line change
`@@ -377,7 +377,7 @@ add_additional_variables(char *name, bool insert)`
`377`	`377`	`{`
`378`	`378`	`struct cursor *ptr;`
`379`	`379`	`struct arguments *p;`
`380`		`- int (* strcmp_fn)(const char , const char ) = (name[0] == ':' ? strcmp : pg_strcasecmp);`
	`380`	`+ int (* strcmp_fn)(const char , const char ) = ((name[0] == ':' \|\| name[0] == '"') ? strcmp : pg_strcasecmp);`
`381`	`381`
`382`	`382`	`for (ptr = cur; ptr != NULL; ptr=ptr->next)`
`383`	`383`	`{`
Original file line number	Diff line number	Diff line change
`@@ -289,7 +289,7 @@ ECPGCursorStmt: DECLARE cursor_name cursor_options CURSOR opt_hold FOR prepared`
`289`	`289`	`{`
`290`	`290`	`struct cursor ptr, this;`
`291`	`291`	`char *cursor_marker = $2[0] == ':' ? mm_strdup("$0") : mm_strdup($2);`
`292`		`- int (* strcmp_fn)(const char , const char ) = ($2[0] == ':' ? strcmp : pg_strcasecmp);`
	`292`	`+ int (* strcmp_fn)(const char , const char ) = (($2[0] == ':' \|\| $2[0] == '"') ? strcmp : pg_strcasecmp);`
`293`	`293`	`struct variable thisquery = (struct variable )mm_alloc(sizeof(struct variable));`
`294`	`294`	`const char *con = connection ? connection : "NULL";`
`295`	`295`	`char *comment;`