comp.lang.ada
 help / color / mirror / Atom feed
From: Bill Findlay <yaldnif.w@blueyonder.co.uk>
Subject: Re: GCC 4.8.1 for Mac OS X
Date: Mon, 08 Jul 2013 18:57:15 +0100
Date: 2013-07-08T18:57:15+01:00	[thread overview]
Message-ID: <CE00BC0B.32184%yaldnif.w@blueyonder.co.uk> (raw)
In-Reply-To: ly61wlzf6c.fsf@pushface.org

On 08/07/2013 08:25, in article ly61wlzf6c.fsf@pushface.org, "Simon Wright"
<simon@pushface.org> wrote:

> Bill Findlay <yaldnif.w@blueyonder.co.uk> writes:
> 
>> On 07/07/2013 19:37, in article lya9lyz06p.fsf@pushface.org, "Simon Wright"
>> <simon@pushface.org> wrote:
>> 
>>> You can find this at
>>> https://sourceforge.net/projects/gnuada/files/GNAT_GCC%20Mac%20OS%20X/4.8.1/
>>> 
>> 
>> Many thanks for that, Simon.
> 
> You're welcome.
> 
>> I see it shares with GNAT GPL 2013 this foible:
> 
> Foible! excellent word.

8-)

>>    h     : hash_code := 0;
> 
> Should that 0 be empty_hash_code?

I can see why you ask that.  empty_hash_code is chosen by the user to
indicate an empty table slot and the function is embedded in a generic
package having empty_hash_code as a constant parameter of the instantiation.
So it might not be zero, and I don't see much reason to start h off at
anything other than 0.

>>    for s of slice loop
>>    exit when s = all_blanks_code;
>>              |
>>>>> warning: variable "s" is not modified in loop body [enabled by default]
>>>>> warning: possible infinite loop [enabled by default]
> 
> Some sort of mistake about the flow analysis, I guess.
> 

Looks like it.

>>       h := rotate_left(h, 2) + s;
>>    end loop;

-- 
Bill Findlay
with blueyonder.co.uk;
use  surname & forename;


  reply	other threads:[~2013-07-08 17:57 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-07 18:37 GCC 4.8.1 for Mac OS X Simon Wright
2013-07-07 22:02 ` Bill Findlay
2013-07-08  7:25   ` Simon Wright
2013-07-08 17:57     ` Bill Findlay [this message]
2013-07-08 20:06 ` Felix Krause
2013-07-08 22:37   ` Simon Wright
2013-07-09 15:31     ` Felix Krause
2013-07-09 19:35       ` Simon Wright
2013-07-17 19:57 ` Felix Krause
2013-07-17 21:00   ` Simon Wright
2013-07-18 20:23     ` Simon Wright
2013-07-21  0:59 ` Michael Rohan
2013-07-21  1:06   ` Bill Findlay
2013-07-21  1:19     ` Michael Rohan
2013-07-21  3:57     ` Dennis Lee Bieber
2013-07-21 11:55   ` Simon Wright
2013-07-21 18:01     ` Michael Rohan
replies disabled

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox