[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-gnubg] gcc 4.0 patch for 0.14.3

From: Russ Allbery
Subject: Re: [Bug-gnubg] gcc 4.0 patch for 0.14.3
Date: Sat, 21 Jan 2006 16:40:56 -0800
User-agent: Gnus/5.110004 (No Gnus v0.4) XEmacs/21.4.17 (linux)

Øystein Johansen <address@hidden> writes:

> At last I was able to look at this. No backgammon playing today. Just a
> lazy wednesday. Really good!

Sounds excellent.  :)  Sorry about the delay in responding; it's been a
busy week.

> This snip of code has never been compiled at my system since it's now
> inside a #if HAVE_MMAP && ! USE_SSE_VECTORIZE and of course I always use
> the vectorization code. The patch is commited.


>> --- gnubg-0.14.3.orig/rollout.c
>> +++ gnubg-0.14.3/rollout.c
>> @@ -219,7 +219,7 @@
>>      nPermutationSeed = n;
>>  }
>> -static int nSkip;
>> +int nSkip;

> Why this? I don't get any errors or warnings with my gcc-4.0.2

In 0.14.3, rollout.h has "extern int nSkip" and rollout.c includes
rollout.h.  A variable can't be declared static after being declared
extern, so this breaks compilation.

My guess is that this has already been fixed some other way in CVS.

> I really recommend trying a cvs build. Lots of things has been improved
> since 0.14.3. Among those things are the sse vectorization which
> increases the evaluation speed by about 50%.

Do you have an estimate for 0.15?  While sometimes Debian does include CVS
snapshots of software, generally we prefer to use official upstream
releases for various reasons.

gnubg is a bit of a special case since it's generally beta software
anyway, but still, if what's in CVS is significantly better, I encourage
you to stick a version number on it!  :)

Russ Allbery (address@hidden)             <http://www.eyrie.org/~eagle/>

reply via email to

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