[Freeciv-Dev] Re: [patches] freeing NULL ptrs

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Sun, Aug 19, 2001 at 01:09:27PM -0400, Ross W. Wetmore wrote:
> assert is a void function and so can only be put in code blocks as
> opposed to expressions. But a divide by zero can do the same thing
> in an expression.

I know.

> These should be tied to NDEBUG to stop the proliferation of control
> variables. NDEBUG should be set on release versions and friendlier
> actions taken as alternate codepaths.

Ahh I didn't know that NDEBUG is set at certain times. From
configure.in:

if test "x$enable_debug" = "xyes"; then
  test "$cflags_set" = set || CFLAGS="$CFLAGS -g"
  CPPFLAGS="$CPPFLAGS -DDEBUG"
else
  if test "x$enable_debug" = "xno"; then
    CPPFLAGS="$CPPFLAGS -DNDEBUG"
  fi
fi

> I.E. the solution is already there and just needs to be used.

You are right.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx