| From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
|---|---|
| To: | Andres Freund <andres(at)2ndquadrant(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [COMMITTERS] pgsql: Add noreturn attributes to some error reporting functions |
| Date: | 2013-02-13 16:08:56 |
| Message-ID: | 511BBA98.8030004@gmx.net |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
On 2/12/13 7:19 AM, Andres Freund wrote:
> On 2013-02-12 12:14:06 +0000, Peter Eisentraut wrote:
>> Add noreturn attributes to some error reporting functions
>
> I wonder if its time to add a macro for this instead of slapping
> __attribute__((noreturn)) everywhere. That way msvc had a chance of
> adding __declspec(noreturn) or whatever its magic incatation is
> transparently. And perhaps other compilers in the future.
Sure, if someone who has access to both msvc and gcc wanted to put in
the leg work, it would be worth a try.
One possible problem is whether the placement of these modifiers is the
same across all potential compilers. But we currently use these in
enough places that it would be easy to find that out by just converting
everything and building it.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2013-02-13 17:48:45 | pgsql: Fix bogus when-to-deregister-from-listener-array logic. |
| Previous Message | Heikki Linnakangas | 2013-02-13 15:53:05 | pgsql: Update visibility map in the second phase of vacuum. |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Atri Sharma | 2013-02-13 16:10:25 | Re: Fractal tree indexing |
| Previous Message | Heikki Linnakangas | 2013-02-13 16:02:43 | Re: Fractal tree indexing |