[Freeciv-Dev] Re: PATCH: Add ai_manage_diplomat().

[Tony Stuckey <stuckey@xxxxxxxxxxxxxxxxx>]

On Mon, Jan 31, 2000 at 10:22:37PM -0500, Greg Wooledge wrote:
> Tony Stuckey (stuckey@xxxxxxxxxxxxxxxxx) wrote:
> Compiler warning here with gcc 2.95.2 -- may be used uninitialized.  I
> looked at the code briefly and the warning appears to be irrelevant as
> long as game.nplayers > 0.... :-)  To shut gcc up, you might want to
> initialize this to NULL or something.

        Yeah, I saw that, and decided GCC was just being a twit.

> I tested this a bit, and the AI actually bribed one of my units (a
> Musketeer fortified outside his city).  It was great!
> 
> Then a couple turns later it crashed:
> 
> 1: Bribe cost -1 in diplomat_bribe by Erik
> > Segmentation fault

        Do you have a saved game?

> Next time I test this I'll run it under gdb.  (Why oh why doesn't the
> civserver leave a core file?!?  At least in the CVS versions, that would
> be tremendously useful....)

        Several OS distributions nowadays default to not allowing core
dumps.  Check what ulimit says from your shell.
[stuckey@jaka stuckey]$ ulimit -a
core file size (blocks)  1000000

        While I haven't seen a core file because of this patch, I do get
them from time to time under Solaris 7.  The one I got this week is in the
revised vsnprintf code while making a historical report.
-- 
Anthony J. Stuckey                              stuckey@xxxxxxxxxxxxxxxxx
"And they said work hard, and die suddenly, because it's fun."
        -Robyn Hitchcock.