On Sat, 26 Apr 2008 11:40:43 -0700
Mark Glines (via RT) <[EMAIL PROTECTED]> wrote:

> #   at t/op/sysinfo.t line 53.
> #          got: '8'
> #     expected: '4'

Found it.  The test was checking the op's report of sizeof(INTVAL)
against $PConfig{intsize}, but INTVAL isn't necessarily defined to
"int".  On linux/amd64, INTVAL is typedeffed to long (via Parrot_Int).

PConfig{intsize} is sizeof(int).
PConfig{intvalsize} is sizeof(INTVAL).  That's the one we want.

Fixed in r27196.

There's an XXX comment in this test for a good reason: We probably also
want a PConfig{floatvalsize}, distinct from PConfig{doublesize}.
Probably not a big deal if Parrot_Float is always declared as a
"double".

Mark

Reply via email to