ypsilon-compat check-char** is not 64-bit safe

hyln9 / ikarus

Optimizing incremental native-code compiler for R6RS scheme. This is a forked repository.

https://launchpad.net/ikarus

Other

5 stars 0 forks source link

Closed hyln9 closed 10 years ago

hyln9 commented 10 years ago

check-char\ currently fills its malloced array using pointer-set-c-int!, which only sets 4 of 8 bytes on 64-bit platforms (at least linux x86_64).

Launchpad Details: #LP360310 Tristan Ravitch - 2009-04-12 23:19:40 -0400

hyln9 commented 10 years ago

Will fix. Thanks.

Launchpad Details: #LPC Abdulaziz Ghuloum - 2009-04-13 03:22:14 -0400

hyln9 commented 10 years ago

I think the patch is wrong. Can you check if the attached patch is correct.

Launchpad Details: #LPC Abdulaziz Ghuloum - 2009-04-13 03:29:24 -0400

hyln9 commented 10 years ago

Thanks, that works.

Launchpad Details: #LPC Tristan Ravitch - 2009-04-13 09:07:22 -0400

hyln9 commented 10 years ago

Fixed in revision 1767. Thanks.

Launchpad Details: #LPC Abdulaziz Ghuloum - 2009-04-13 09:45:46 -0400