| From: | RekGRpth <rekgrpth(at)gmail(dot)com> |
|---|---|
| To: | Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at> |
| Cc: | 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-06 16:41:00 |
| Message-ID: | CAPgh2mKi8t8T7Tsqx-EwSMYG_xYQQSY30qp18sH8dVySP4Fmdg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs |
Fixed!
ср, 6 дек. 2023 г. в 21:38, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>:
>
> On Wed, 2023-12-06 at 19:44 +0500, RekGRpth wrote:
> > something like?
>
> Yes, perhaps - although the variable name "tzp" = "time zone pointer"
> is not appropriate any more.
>
> Perhaps the "tzp" variable could be left as it is, and only the
> comparisons "if (tzp == NULL)" be removed.
>
> Yours,
> Laurenz Albe
>
| Attachment | Content-Type | Size |
|---|---|---|
| tzp2.diff | text/x-patch | 2.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Laurenz Albe | 2023-12-06 21:01:58 | Re: BUG #18230: Redundant comparison of a local variable 'tzp' address with a NULL value at dt_common.c |
| Previous Message | Laurenz Albe | 2023-12-06 16:38:02 | Re: BUG #18230: Redundant comparison of a local variable 'tzp' address with a NULL value at dt_common.c |