Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#557 closed defect (fixed)

pretty-print issue on master

Reported by: Mario Domenech Goulart Owned by: felix winkelmann
Priority: critical Milestone: 4.9.0
Component: core libraries Version: 4.6.x
Keywords: pp, pretty-print Cc:
Estimated difficulty:

Description

It seems that the current master code for pretty-print is slightly broken.

Here's a simple test case:

(pp "\\\"")

4.6.3 prints "\\\"", while master prints "\\"".

Attachments (2)

pp-fix.diff (2.1 KB) - added by Mario Domenech Goulart 10 years ago.
Proposed fix to pretty-print
pp-test.scm (299 bytes) - added by Mario Domenech Goulart 10 years ago.
Some test cases

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by Mario Domenech Goulart

Attachment: pp-fix.diff added

Proposed fix to pretty-print

Changed 10 years ago by Mario Domenech Goulart

Attachment: pp-test.scm added

Some test cases

comment:1 Changed 10 years ago by Mario Domenech Goulart

I've attached a patch which seems to fix the problem and a file with some test cases (using the behavior of 4.6.3 as reference).

comment:2 Changed 10 years ago by felix winkelmann

Resolution: fixed
Status: newclosed

comment:3 Changed 10 years ago by felix winkelmann

Milestone: 4.7.04.8.0

Milestone 4.7.0 deleted

comment:4 Changed 9 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.