Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2003:
[Freeciv-Dev] Re: core file on civserver, http://civserver.freeciv.org/g
Home

[Freeciv-Dev] Re: core file on civserver, http://civserver.freeciv.org/g

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Per I. Mathisen" <per@xxxxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: core file on civserver, http://civserver.freeciv.org/games/162970
From: Raimar Falke <i-freeciv-lists@xxxxxxxxxxxxx>
Date: Sun, 30 Nov 2003 14:30:04 +0100

On Sat, Nov 29, 2003 at 11:33:36PM +0000, Per I. Mathisen wrote:
> On Sat, 29 Nov 2003 freeciv@xxxxxxxxxxxxxxxxxxxxx wrote:
> > #20x40076364 in abort () from /lib/libc.so.6
> > #30x4006f15d in __assert_fail () from /lib/libc.so.6
> > #40x0808f47f in check_units () at sanitycheck.c:239
> > #50x0808f6b4 in sanity_check () at sanitycheck.c:285
> > #60x0805091d in main_loop () at srv_main.c:1792
> 
> Is there actually any point in doing asserts in sanity_check()? 

> The backtraces generated from such asserts are usually useless

I agree.

> , while the errors are often recoverable if they didn't assert. Much
> better to output some useful error logging, IMHO.

We had this discussion (die or recover) in the past. I'm against doing
this in CVS HEAD but it may possible to apply such patch to pubserver.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "The Internet is really just a series of bottlenecks 
  joined by high speed networks."
    -- Sam Wilson


[Prev in Thread] Current Thread [Next in Thread]