Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#54 closed defect (fixed)

pointer-s8-ref should explicitly use signed char

Reported by: sjamaan Owned by: felix winkelmann
Priority: critical Milestone:
Component: core libraries Version: 4.0.x
Keywords: pointer, signedness, platform dependent Cc:
Estimated difficulty:

Description

On some architectures (PowerPC, ARM), "char" is unsigned by default. On these platforms, the lolevel tests fail. This can be fixed simply by adding the "signed" qualifier:

(define pointer-s8-ref
  (getter-with-setter
     (foreign-lambda* int ([c-pointer p]) "return(*((signed char *)p));")
       pointer-s8-set!) )

Change History (2)

comment:1 Changed 10 years ago by felix winkelmann

Resolution: fixed
Status: newclosed

fixed in r15169. Thanks.

comment:2 Changed 10 years ago by (none)

Milestone: 4.1.0

Milestone 4.1.0 deleted

Note: See TracTickets for help on using tickets.