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