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

["Ross W. Wetmore" <rwetmore@xxxxxxxxxxxx>]

Fixing the conditional was my quick hack too ...

Cheers,
RossW
=====

At 02:58 PM 02/01/17 +0000, Gregory Berkolaiko wrote:
> --- 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
>
>