Commitd19439f

Thomas G. Lockhart

committed

Remove the definition for set_name_needs_quotes() on the assumption that

it is now obsolete. Need some regression test cases to prove otherwise...

1 parentb875b59 commitd19439fCopy full SHA for d19439f

File tree

-23

lines changed

-23

lines changed

Lines changed: 1 addition & 23 deletions

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`*`
`12`	`12`	`*`
`13`	`13`	`* IDENTIFICATION`
`14`		`- * $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.307 2002/04/2119:21:49 thomas Exp $`
	`14`	`+ * $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.308 2002/04/21 21:37:03 thomas Exp $`
`15`	`15`	`*`
`16`	`16`	`* HISTORY`
`17`	`17`	`* AUTHORDATEMAJOR EVENT`
`@@ -6676,25 +6676,3 @@ doNegateFloat(Value *v)`
`6676`	`6676`	`v->val.str = newval;`
`6677`	`6677`	`}`
`6678`	`6678`	`}`
`6679`		`-`
`6680`		`-/*`
`6681`		`- * Decide whether to put double quotes around a name appearing in a SET`
`6682`		`- * name_list. Presently, do so if the name contains whitespace, commas,`
`6683`		`- * or uppercase characters. (This is correct assuming that the result`
`6684`		`- * will be deparsed by SplitIdentifierString or similar logic.)`
`6685`		`-*/`
`6686`		`-staticbool`
`6687`		`-set_name_needs_quotes(constchar *name)`
`6688`		`-{`
`6689`		`-if (*name =='\0')`
`6690`		`-returntrue;/* empty name does need quotes*/`
`6691`		`-while (*name)`
`6692`		`-{`
`6693`		`-if (*name ==',' \|\|`
`6694`		`-isspace((unsignedchar) *name) \|\|`
`6695`		`-isupper((unsignedchar) *name))`
`6696`		`-returntrue;`
`6697`		`-name++;`
`6698`		`-}`
`6699`		`-returnfalse;`
`6700`		`-}`

Comments

(0)