Re: pgsql: A few fixups in error handling: mark pg_re_throw() as noreturn

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: A few fixups in error handling: mark pg_re_throw() as noreturn
Date: 2007-05-04 08:01:41
Message-ID: 20070504080141.GE29747@svr2.hagander.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Do we want to add other compiler specific fixes, such as the MSVC one?
Should be easy enough to do if we want to...

//Magnus

On Thu, May 03, 2007 at 11:01:02PM -0300, Tom Lane wrote:
> Log Message:
> -----------
> A few fixups in error handling: mark pg_re_throw() as noreturn for gcc,
> and for other compilers, insert a dummy exit() call so that they understand
> PG_RE_THROW() doesn't return. Insert fflush(stderr) in ExceptionalCondition,
> per recent buildfarm evidence that that might not happen automatically on some
> platforms. And const-ify ExceptionalCondition's declaration while at it.
>
> Modified Files:
> --------------
> pgsql/src/backend/utils/error:
> assert.c (r1.33 -> r1.34)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/error/assert.c.diff?r1=1.33&r2=1.34)
> elog.c (r1.184 -> r1.185)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/error/elog.c.diff?r1=1.184&r2=1.185)
> pgsql/src/include:
> postgres.h (r1.79 -> r1.80)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/postgres.h.diff?r1=1.79&r2=1.80)
> pgsql/src/include/utils:
> elog.h (r1.85 -> r1.86)
> (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/utils/elog.h.diff?r1=1.85&r2=1.86)
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: You can help support the PostgreSQL project by donating at
>
> http://www.postgresql.org/about/donate

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message User Dpage 2007-05-04 08:10:45 stackbuilder - wizard: New Directory
Previous Message Magnus Hagander 2007-05-04 08:01:07 Re: pgsql: Teach tuplesort.c about "top N" sorting, in which only the first