History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: BOO-889
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Marcus Griep
Reporter: Marcus Griep
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Boo

BooPrinterVistor makes ugly elif chains

Created: 06/Oct/07 09:33 PM   Updated: 09/Oct/07 01:33 PM
Component/s: Compiler
Affects Version/s: 0.7.9
Fix Version/s: 0.8

Time Tracking:
Not Specified

File Attachments: 1. Text File correct-elif-patch.patch (3 kb)
2. Text File elifprinter.patch (0.8 kb)


Patch Submitted: Yes


 Description  « Hide
Right now, the printer visitor doesn't know about elif statements. The attached patch helps the printer to notice elif statements and print them accordingly.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Rodrigo B. de Oliveira - 09/Oct/07 01:32 PM
Complete version of the patch I've actually applied.

The original patch was missing test case corrections and it actually had a bug when dealing with if statements with elifs but no else.


Rodrigo B. de Oliveira - 09/Oct/07 01:33 PM
Thanks for your patch but I had to refuse it. Review the patch I attached to this issue for details.

Fixed in the repository anyway.