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

[Russ Allbery]

Ø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.

Thanks!

>> --- 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/>