| Summary: | the 'else' keyword is not being recognized | ||
|---|---|---|---|
| Product: | [Applications] kturtle | Reporter: | Jarav <jagrav> |
| Component: | general | Assignee: | Cies Breijs <cies> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | 0.6 | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: |
trace file
fixes the else issue so the doc examples work |
||
|
Description
Jarav
2005-11-02 19:40:31 UTC
Reproducable. Important statement to make working programs. Created attachment 14029 [details]
trace file
Trace file using following example:
reset
a=5
if a==4 [
print "ok"
]
else [
print "not ok"
]
Confirmed on kturtle 0.6 on SuSE 10. I've tried copying code in verbatim from the manual, so this is IMHO a serious issue, since it limits the logo implementation. Should be fixed in next version. *** Bug has been marked as fixed ***. Bbefore the fix only ] else [ worked. Now ] else [ works but not ] else [ as in the manual examples. I reopen the bug report so that the examples in the user manual can be compiled. I'll add a patch, to be committed whenever possible as 3.5.6 tagging has just been done. Created attachment 19301 [details]
fixes the else issue so the doc examples work
SVN commit 676365 by annma:
fix "else" bug
BUG=115555
M +1 -1 parser.cpp
--- branches/KDE/3.5/kdeedu/kturtle/src/parser.cpp #676364:676365
@@ -932,7 +932,7 @@
if (currentToken.type == tokElse) // else part
{
matchToken(tokElse);
-
+ while (currentToken.type == tokEOL) getToken();
if (currentToken.type == tokDo) getToken(); // next word
if(currentToken.type == tokBegin) node->appendChild( Block() ); // else is followed by block
|