diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2002-03-20 19:45:13 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2002-03-20 19:45:13 +0000 |
commit | 337b22cb473f1c5cca011a511c488d20e153eec4 (patch) | |
tree | bfec217a7ac7672d796217bfd9dce348a255e1b0 /src/backend/parser/parse_func.c | |
parent | 251282d4b7bf7593cece7c4ce5669beb778604e3 (diff) | |
download | postgresql-337b22cb473f1c5cca011a511c488d20e153eec4.tar.gz postgresql-337b22cb473f1c5cca011a511c488d20e153eec4.zip |
Code review for DOMAIN patch.
Diffstat (limited to 'src/backend/parser/parse_func.c')
-rw-r--r-- | src/backend/parser/parse_func.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/src/backend/parser/parse_func.c b/src/backend/parser/parse_func.c index ed39d6c1036..6dc7b440fa8 100644 --- a/src/backend/parser/parse_func.c +++ b/src/backend/parser/parse_func.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.117 2002/03/12 00:51:55 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/parser/parse_func.c,v 1.118 2002/03/20 19:44:29 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -575,8 +575,7 @@ agg_select_candidate(Oid typeid, CandidateList candidates) { current_typeid = current_candidate->args[0]; - if (current_typeid == typeid - || IS_BINARY_COMPATIBLE(current_typeid, typeid)) + if (IsBinaryCompatible(current_typeid, typeid)) { last_candidate = current_candidate; ncandidates++; @@ -815,9 +814,7 @@ func_select_candidate(int nargs, { if (input_typeids[i] != UNKNOWNOID) { - if (current_typeids[i] == input_typeids[i] || - IS_BINARY_COMPATIBLE(current_typeids[i], - input_typeids[i])) + if (IsBinaryCompatible(current_typeids[i], input_typeids[i])) nmatch++; } } @@ -1115,8 +1112,7 @@ func_get_detail(char *funcname, Node *arg1 = lfirst(fargs); if ((sourceType == UNKNOWNOID && IsA(arg1, Const)) || - sourceType == targetType || - IS_BINARY_COMPATIBLE(sourceType, targetType)) + IsBinaryCompatible(sourceType, targetType)) { /* Yup, it's a type coercion */ *funcid = InvalidOid; |