Get rid of the pre-C11 _Alignof define in zic.c

Started by Bertrand Drouvot2 months ago2 messageshackers
Jump to latest
#1Bertrand Drouvot
bertranddrouvot.pg@gmail.com

Hi hackers,

Now that f5e0186f865c is in, I think that we can get rid of this pre-C11 define
and just rely on the C11 _Alignof definition. That's done in the attached.

Note that e6be84356bbb already introduced the usage of _Noreturn in it.

Regards,

--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com

Attachments:

v1-0001-Get-rid-of-pre-C11-_Alignof-define-in-zic.c.patchtext/x-diff; charset=us-asciiDownload+0-6
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bertrand Drouvot (#1)
Re: Get rid of the pre-C11 _Alignof define in zic.c

Bertrand Drouvot <bertranddrouvot.pg@gmail.com> writes:

Now that f5e0186f865c is in, I think that we can get rid of this pre-C11 define
and just rely on the C11 _Alignof definition. That's done in the attached.

I'm not in favor of introducing random discrepancies from tzdb
upstream.

(We are well overdue for re-syncing this code with recent tzdb,
but I don't think changes like this one will make that any less
painful.)

regards, tom lane