Re: [Axiom-developer] Broken hypertex

[Gabriel Dos Reis]

"Bill Page" <address@hidden> writes:

| On October 13, 2006 8:55 PM Gabriel Dos Reis wrote:
| > 
| > Waldek Hebisch <address@hidden> writes:
| > 
| > | Gabriel Dos Reis wrote:
| > | > Waldek Hebisch <address@hidden> writes:
| > | > 
| > | > | Gabriel Dos Reis wrote:
| > | > | > Waldek Hebisch <address@hidden> writes:
| > | > | > 
| > | > | > | Playing with build-improvements I have noticed that 
| > | > | > | hypertex is broken:
| > | > | > 
| > | > | > Is that reproducible with silver?
| > | > | > 
| > | > | 
| > | > | Probably no: when I tried build-improvements with 
| > | > | gcl-2.6.7 the problem went away.
| > | > 
| > | > Interesting.
| > | > 
| > | > We have a case to investigate.  I seem to remember, but 
| > | > I'm not sure, that the socket interface may have changed
| > | > from 2.6.7 to 2.6.8pre. Bill?
| 
| The change in the socket interface is due only to the better
| treatment of data types in 2.6.8pre and should not be the
| source of any functional changes.

That is good to know; thanks!

[...]

| > It is important that as we work on "fixing" things, we do
| > clearly understand what we are fixing.
| > 
| 
| I agree. I think this thread could benefit by a more explicit
| reference to the source code. See below.

[...]

| > 
| > Exactly, a typo (& instead of %), no ? I think Camm read this
| > mailing list but writing directly to him would be a good idea.
| > 
| 
| I agree. The following line in the diff output below
| 
| +  if (x->st.st_dim == leng && leng & sizeof(object)
| 
| should actually read
| 
| +  if (x->st.st_dim == leng && leng % sizeof(object)

I just updated my version of gcl-2.6.8pre and saw that the typo is
still there.  I'll check in a fix later, unless Camm beats me at it.

-- Gaby