| From: | jian he <jian(dot)universality(at)gmail(dot)com> |
|---|---|
| To: | Kim Johan Andersson <kimjand(at)kimmet(dot)dk> |
| Cc: | Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: [PATCH] Add support function for containment operators |
| Date: | 2023-12-16 13:03:00 |
| Message-ID: | CACJufxHO4nwfu8GRwKpbA31sDbe9inJk7p0hsxGtiChNmSp6VA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
fix a typo and also did a minor change.
from
+ if (lowerExpr != NULL && upperExpr != NULL)
+ return (Node *) makeBoolExpr(AND_EXPR, list_make2(lowerExpr, upperExpr),
-1);
+ else if (lowerExpr != NULL)
+ return (Node *) lowerExpr;
+ else if (upperExpr != NULL)
+ return (Node *) upperExpr;
to
+ if (lowerExpr != NULL && upperExpr != NULL)
+ return (Node *) makeBoolExpr(AND_EXPR, list_make2(lowerExpr, upperExpr),
-1);
+ else if (lowerExpr != NULL)
+ return (Node *) lowerExpr;
+ else if (upperExpr != NULL)
+ return (Node *) upperExpr;
+ else
+ {
+ Assert(false);
+ return NULL;
+ }
because cfbot says:
15:04:38.116] make -s -j${BUILD_JOBS} clean
[15:04:38.510] time make -s -j${BUILD_JOBS} world-bin
[15:04:43.272] rangetypes.c:2908:1: error: non-void function does not
return a value in all control paths [-Werror,-Wreturn-type]
[15:04:43.272] }
[15:04:43.272] ^
[15:04:43.272] 1 error generated.
also add some commit messages, I hope it will be useful.
| Attachment | Content-Type | Size |
|---|---|---|
| v5-0001-Simplify-containment-in-range-constants-with-supp.patch | text/x-patch | 20.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alena Rybakina | 2023-12-16 15:41:27 | Re: [PoC] Reducing planning time when tables have many partitions |
| Previous Message | Andres Freund | 2023-12-16 12:29:00 | Re: Clang optimiser vs preproc.c |