Re: mcxt.c - Mailing list pgsql-patches

From Gaetano Mendola
Subject Re: mcxt.c
Date
Msg-id 00f001c3764e$26430940$4c720b3e@mm.eutelsat.org
Whole thread Raw
In response to mcxt.c  ("Mendola Gaetano" <mendola@bigfoot.com>)
List pgsql-patches
"Neil Conway" <neilc@samurai.com> wrote:
> On Mon, 2003-09-08 at 11:09, Gaetano Mendola wrote:
> > "Tom Lane" <tgl@sss.pgh.pa.us> wrote:
> > > I see no value at all in an assert.  The code will dump core just fine
> > > with or without an assert ...
> >
> > Right but an assert can display information about the file and line
number
> > without debug the application
>
> I think the percentage of deployments that enable assertions (which
> causes a runtime performance hit) but NOT debugging info (which does
> not) is pretty small.
>
> ISTM that checking for non-NULL pointers is pretty pointless: just
> because a pointer happens to be non-NULL doesn't mean it is any more
> valid, and dereferencing a NULL pointer is easy enough to track down in
> any case.

I'm not speaking only about to test if pointer is null or not but also do
some
assert to better understand wich condition shall be verified in some part of
the
code also to have clear what is going on inside code that may be after years
and years is not clear anymore.

May be I'm not clear enough.
Please tell me when and were an assert shall be used.

Regards
Gaetano Mendola






pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: ISO 8601 "Time Intervals" of the "format with time-unit
Next
From: Tom Lane
Date:
Subject: Re: ISO 8601 "Time Intervals" of the "format with time-unit deignators"