Allow CASE statement to contain *only* untyped result clauses or nulls.

Thomas G. Lockhart · Thomas G. Lockhart · commit 400676afc9ca · 1999-09-13T04:21:21.000Z
Almost worked before, but forgot one place to check.
 Reported by Tatsuo Ishii.
Still does not do the right thing if inserting into a non-string target
 column. Should look for a type coersion later, but doesn't.
diff --git a/src/backend/parser/parse_expr.c b/src/backend/parser/parse_expr.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.50 1999/07/11 02:04:19 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.50.2.1 1999/09/13 04:21:21 thomas Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -467,7 +467,8 @@ transformExpr(ParseState *pstate, Node *expr, int precedence)
 					 * only bother with conversion if not NULL and
 					 * different type...
 					 */
-					if (wtype && (wtype != ptype))
+					if (wtype && (wtype != UNKNOWNOID)
+						&& (wtype != ptype))
 					{
 						if (can_coerce_type(1, &wtype, &ptype))
 						{

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`*`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.50 1999/07/11 02:04:19 tgl Exp $`
	`10`	`+ * $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.50.2.1 1999/09/13 04:21:21 thomas Exp $`
`11`	`11`	`*`
`12`	`12`	`*-------------------------------------------------------------------------`
`13`	`13`	`*/`
`@@ -467,7 +467,8 @@ transformExpr(ParseState pstate, Node expr, int precedence)`
`467`	`467`	`* only bother with conversion if not NULL and`
`468`	`468`	`* different type...`
`469`	`469`	`*/`
`470`		`- if (wtype && (wtype != ptype))`
	`470`	`+ if (wtype && (wtype != UNKNOWNOID)`
	`471`	`+ && (wtype != ptype))`
`471`	`472`	`{`
`472`	`473`	`if (can_coerce_type(1, &wtype, &ptype))`
`473`	`474`	`{`