From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
Cc: | Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Maxim Orlov <orlovmg(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Custom tuplesorts for extensions |
Date: | 2022-07-24 23:23:05 |
Message-ID: | 20220724232305.GH7789@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Note that 0001+0002 (without the others) incurs warnings:
$ time { make -j4 clean; make -j4; } >/dev/null
tuplesort.c:1883:9: warning: unused variable 'i' [-Wunused-variable]
tuplesort.c:1955:10: warning: unused variable 'i' [-Wunused-variable]
tuplesort.c:2026:9: warning: unused variable 'i' [-Wunused-variable]
tuplesort.c:2103:10: warning: unused variable 'i' [-Wunused-variable]
(I wondered in the past if cfbot should try to test for clean builds of subsets
of patchsets, and it came up recently with the JSON patches.)
Also, this comment has some bad indentation:
* Set state to be consistent with never trying abbreviation.
--
Justin
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2022-07-25 01:08:42 | Re: Refactor to make use of a common function for GetSubscriptionRelations and GetSubscriptionNotReadyRelations. |
Previous Message | Kenaniah Cerny | 2022-07-24 23:03:17 | Re: Proposal: allow database-specific role memberships |