From: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: simplehash.h comment |
Date: | 2018-08-28 00:36:50 |
Message-ID: | CAEepm=02Vc9P9zPO5b9R9SieQBDaiMqtWSUf4sHvgeJj-f-RXA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Aug 28, 2018 at 1:40 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> writes:
> > ... and a patch. Thoughts?
>
> It's not real obvious why you're adding inclusion guards over just a
> subset of the header. A short comment about that would be worthwhile.
Right, thanks. I added a comment and committed.
Obviously the longer term fix for that particular issue is not to
define those functions in that header at all. If no one beats me to
it, I'd like to have another look at consolidating all our popcount()
and ffs() functions[1], and this fls()/log2(), pow2() stuff seems like
it might be part of the same refactoring expedition. For now I just
added the missing include guards to unblock some other stuff I'm about
to propose that wants to use simplehash.h.
--
Thomas Munro
http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Wood | 2018-08-28 00:39:36 | First steps to being a contributer |
Previous Message | Bradley DeJong | 2018-08-27 22:19:49 | Re[2]: doc - improve description of default privileges |