[CLJ-346] (pprint-newline :fill) is not handled correctly Created: 12/May/10 Updated: 29/Nov/10 |
|
| Status: | Open |
| Project: | Clojure |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Backlog |
| Type: | Defect | Priority: | Minor |
| Reporter: | Assembla Importer | Assignee: | Tom Faulhaber |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Approval: | Incomplete |
| Waiting On: | Tom Faulhaber |
| Description |
|
Filled pretty printing (where we try to fit as many elements on a line as possible) is being too aggressive as we can see when we try to print the following array: user> (binding [*print-right-margin* 20] (pprint (int-array (range 10)))) Produces: [0, Rather than [0, 1, 2, 3, 4, or something like that. (I haven't worked through the exact correct representation for this case). We currently only use :fill style newlines for native java arrays. |
| Comments |
| Comment by Assembla Importer [ 24/Aug/10 8:01 AM ] |
|
Converted from http://www.assembla.com/spaces/clojure/tickets/346 |
| Comment by Assembla Importer [ 24/Aug/10 8:01 AM ] |
|
stu said: [file:diLxv6y4Sr35GVeJe5cbLr] |
| Comment by Assembla Importer [ 24/Aug/10 8:01 AM ] |
|
stu said: The second patch includes the first, and adds another test. |
| Comment by Assembla Importer [ 24/Aug/10 8:01 AM ] |
|
tomfaulhaber said: This patch was attached to the wrong bug. It should be attached to bug #347. There is no fix for this bug yet. |
| Comment by Rich Hickey [ 05/Nov/10 8:07 AM ] |
|
Is this current? |
| Comment by Stuart Halloway [ 29/Nov/10 8:48 PM ] |
|
Tom, this patch doesn't apply, and I am not sure why. Can you take a look? |