[#BOO-889] BooPrinterVistor makes ugly elif chains

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:

correct-elif-patch.patch (3 kb)
2.

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

[ Permlink | « Hide ]

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.

[ Permlink | « Hide ]

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.