| From: | RekGRpth <rekgrpth(at)gmail(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | Richard Guo <guofenglinux(at)gmail(dot)com>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-bugs(at)lists(dot)postgresql(dot)org |
| Subject: | Re: BUG #18230: Redundant comparison of a local variable 'tzp' address with a NULL value at dt_common.c |
| Date: | 2023-12-07 02:39:35 |
| Message-ID: | CAPgh2mJYYbjqk-Qk8mRGTPO1bWnkGngQf1BDucNUYNN9hoDWmw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Fixed!
чт, 7 дек. 2023 г. в 07:32, Michael Paquier <michael(at)paquier(dot)xyz>:
>
> On Thu, Dec 07, 2023 at 09:51:57AM +0800, Richard Guo wrote:
> > There are several places in DecodeDateTime() where the value of '*tzp'
> > is changed. It seems to me that these assignments are unnecessary since
> > the value of '*tzp' is not utilized anywhere in the code. Can we also
> > remove these assignments?
>
> *tzp could be given to DecodeTimezone() and manipulated inside it, but
> you are right that we don't use it at all in the DecodeDateTime()
> path. Other callers of DecodeTimezone() may depend on the changes
> done inside it, though. Perhaps DecodeTimezone() should be extended
> so as it can accept a NULL value and use that in DecodeDateTime(),
> eliminating the need for *tzp entirely?
> --
> Michael
| Attachment | Content-Type | Size |
|---|---|---|
| rm_tzp.diff | text/x-patch | 4.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrei Lepikhov | 2023-12-07 02:55:08 | Re: BUG #18187: Unexpected error: "variable not found in subplan target lists" triggered by JOIN |
| Previous Message | Michael Paquier | 2023-12-07 02:32:33 | Re: BUG #18230: Redundant comparison of a local variable 'tzp' address with a NULL value at dt_common.c |