pgsql: Add noreturn attributes to some error reporting functions

Started by Peter Eisentrautalmost 13 years ago3 messages
#1Peter Eisentraut
peter_e@gmx.net

Add noreturn attributes to some error reporting functions

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/0cb1fac3b19f01025b63d2cdceabb8767185da28

Modified Files
--------------
contrib/cube/cubescan.l | 2 +-
contrib/seg/segscan.l | 2 +-
src/backend/replication/repl_scanner.l | 2 +-
src/include/mb/pg_wchar.h | 4 ++--
src/include/parser/parse_relation.h | 4 ++--
src/include/storage/lock.h | 2 +-
src/include/tcop/tcopprot.h | 2 +-
src/pl/plpgsql/src/pl_scanner.c | 2 +-
8 files changed, 10 insertions(+), 10 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Andres Freund
andres@2ndquadrant.com
In reply to: Peter Eisentraut (#1)
Re: pgsql: Add noreturn attributes to some error reporting functions

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.

Greetings,

Andres Freund

--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#3Peter Eisentraut
peter_e@gmx.net
In reply to: Andres Freund (#2)
Re: [COMMITTERS] pgsql: Add noreturn attributes to some error reporting functions

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.

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers