-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug in countdown DO +LOOP #12
Comments
Before implementing +LOOP I couldn't find any clear requirements. I then compared different implementations and found one the addressed the risk of very long loops due to erroneous upper limit and starting point. That's the one I implemented, since it doesn't require looking at the sign of the increment. |
How should the following code behave? |
gforth and mecrisp-stellaris say: My understanding is that the TOS index is increasing the test is run against (Equivalently, I think it's This is crappy, b/c https://github.com/gerryjackson/forth2012-test-suite is written for ANS Forth. This is in core.fs:
|
Thanks! Good links, too! I fixed it. If you need binaries right away, I updated the 2.2.6.snapshot. |
Binaries not necessary, and I can confirm that at least the obvious test cases are working as they should. |
should print
4 3 2 1
. Instead, it just prints4
.The text was updated successfully, but these errors were encountered: