pforth icon indicating copy to clipboard operation
pforth copied to clipboard

Published 20 hours ago • verified publisher icon philburk

unit test fails for S", need dual storage

Open philburk opened this issue 3 years ago • 2 comments

I think that S" should use two storage areas to avoid collisions.

INCORRECT RESULT: T{ S" abcd" S" 1234" SSQ13 S= ROT ROT SSQ12 S= -> TRUE TRUE }T

philburk avatar Sep 13 '20 20:09 philburk

Just got this failure message when "make test" under Mac. So this is because there's only one hard drive in the system?

nobodxbodon avatar Jan 03 '22 21:01 nobodxbodon

Just got this failure message when "make test" under Mac. So this is because there's only one hard drive in the system?

No, it is referring to memory storage areas for these strings S" abcd", etc. Sounds like they are currently sharing memory and overwrite eachother's memory when they are used within a word.

catb0t avatar Jan 03 '22 21:01 catb0t

The current S" passes the tests for ANS Forth. https://forth-standard.org/standard/core/Sq

So I don't think it is worth fixing this. I substituted the old tests with the new test from ANS.

philburk avatar Dec 12 '22 00:12 philburk