From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Raising our compiler requirements for 9.6 |
Date: | 2015-08-12 07:29:21 |
Message-ID: | 20150812072921.GC8470@awork2.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2015-08-11 22:34:40 -0400, Noah Misch wrote:
> On Tue, Aug 11, 2015 at 01:04:48PM +0200, Andres Freund wrote:
> > On 2015-08-05 15:46:36 +0200, Andres Freund wrote:
> > > On 2015-08-05 15:08:29 +0200, Andres Freund wrote:
> > > > We might later want to change some of the harder to maintain macros to
> > > > inline functions, but that seems better done separately.
> > >
> > > Here's a conversion for fastgetattr() and heap_getattr()
>
> > Slightly updated version attached.
>
> > In my opinion this drastically increases readability and thus should be
> > applied. Will do so sometime tomorrow unless there's protest.
>
> -1 to introducing more inline functions before committable code replaces what
> you've already pushed for this thread.
Seriously?
I've no problem with "fixing" anything. So far we have don't seem to
have to come to a agreement what exactly that fix would be. Tom has
stated that he doesn't want lock.h made smaller on account of frontend
code including it, and you see that as the right way.
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Moser | 2015-08-12 07:30:06 | Re: How to compare different datums within from a tuple? |
Previous Message | Michael Paquier | 2015-08-12 05:36:43 | TransactionIdGetCommitTsData and its dereferenced pointers |