[Freeciv-Dev] Re: civserver segfault with new research system (PR#1221)

[Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>]

 --- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote: 
> On Wed, Jan 16, 2002 at 09:57:35PM +0100, Christian Knoke wrote:
> >
> > Is this discussion related to *this* bug: !?
> > 
> > > civserver: tech.c:56: set_invention: Zusicherung
> »!is_barbarian(pplayer)«
> > nicht erfüllt.
> > Abgebrochen (core dumped)
> 
> This is a new one (at least for me). It looks like
> game.global_advances really can't be used in leakage style 1.

It can if you remove assert and change
if (value == TECH_KNOWN) {
  game.global_advances[tech]++;
}

to 

if (value == TECH_KNOWN && !is_barbarian(pplayer)) {
  game.global_advances[tech]++;
}

which is i think the right way to be.  Alternatively, count barbarians in
leakage too because this is how the Library works right now.

G.

__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page
from News and Sport to Email and Music Charts
http://uk.my.yahoo.com