small patch re tcl8.1 and more

[Bruce Stephens]

Syd Polk <spolk@redhat.com> writes:


[...]

> >         }
> >       } else {
> > !         Tcl_Obj **objv, *paramObjv[2] = {objPtr1, objPtr2};
> >         int objc;
> > 
> >         /*
> >          * We made space for the two things to compare. Replace them
> >          * and evaluate the result.
> > --- 2856,2866 ----
> >             order = -1;
> >         }
> >       } else {
> > !         Tcl_Obj **objv, *paramObjv[2];
> >         int objc;
> > 
> > +       paramObjv[0] = objPtr1;
> > +       paramObjv[1] = objPtr2;

[...]

> I thought ANSI C allowed this.

I don't think so, I think it's a common extension (supported by gcc
and VC++).

> This might be fixed in the distribution version of Tcl. We are in
> the process of upgrading Tcl versions, so it might not be necessary.

It's fixed in the current CVS.  I guess it's in 8.3, too, but I
haven't checked.

[...]

-- 
Bruce Stephens			Bruce.Stephens@MessagingDirect.com
MessagingDirect(UK) Ltd		<URL: http://www.MessagingDirect.com/ >