[CONTRIB-66] fix pprint tests and reenable Created: 31/Jan/10 Updated: 24/Aug/10 Resolved: 24/Aug/10 |
|
| Status: | Closed |
| Project: | Clojure-Contrib |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Defect | ||
| Reporter: | Anonymous | Assignee: | Tom Faulhaber |
| Resolution: | Completed | Votes: | 0 |
| Labels: | None | ||
| Description |
|
Apparently (according to ticket 65), someone has broken the pprint tests, presumably during the switch to maven and all the hacking to get around gen-class/maven bugs. Take a look at this and fix it. |
| Comments |
| Comment by Assembla Importer [ 24/Aug/10 10:08 PM ] |
|
Converted from http://www.assembla.com/spaces/clojure/tickets/66 |
| Comment by Assembla Importer [ 24/Aug/10 10:08 PM ] |
|
stuart.sierra said: It appears that the ColumnWriter/-init function is getting called with 3 arguments. |
| Comment by Assembla Importer [ 24/Aug/10 10:08 PM ] |
|
stuart.sierra said: I don't think this was caused by changes in the build. Something has changed about how arguments get passed to init functions. The ColumnWriter init function is getting the same arguments as the PrettyWriter init function (a Writer, 72, and 40). |
| Comment by Assembla Importer [ 24/Aug/10 10:08 PM ] |
|
stuart.sierra said: (In revision:5b3beaed9ef8520c8c1e09ff7ac5b817786f51e2) Fix mis-named ns in pprint.gen-class; refs #66 Branch: master |
| Comment by Assembla Importer [ 24/Aug/10 10:08 PM ] |
|
stuart.sierra said: Never mind, it was a simple typo in the gen_class.clj file. Nothing to do with pprint or with the build process. Tests re-enabled; all pass. |