[CLJ-1278] Provide a useful implementation of toString() for Clojure functions Created: 10/Oct/13 Updated: 14/Oct/13
|Affects Version/s:||Release 1.5|
|Reporter:||Howard Lewis Ship||Assignee:||Unassigned|
|Patch:||Code and Test|
Clojure functions do not provide a useful override of the default Java Object.toString() method.
Because of this, any time a Clojure function is printed out, it places a load on the develop to mentally reverse the necessary name mangling to get back to the namespace and function name.
I would vastly prefer better information here: the non-mangled name of the function, including namespace and, ideally, a little bit of the available meta-data: the file name and line number.
In other words, instead of novate.core.processing.async$locate_destination@2690d691 something more like novate.core.processing.async/locate-destination(async.clj:231).
Ideally, anything that is code generated as a class should implement a useful toString(). It would be nice if reified types could identify the containing namespace and function (and file/line number) as part of their default toString().
I suspect this would need to be controlled by a compiler option as it would slightly increase the generated bytecode size, and (imperceptably?) affect compilation speed.
|Comment by Howard Lewis Ship [ 10/Oct/13 8:39 PM ]|
Contains changes and updated tests. I don't have any details on if this affects compiler performance or generated code size in any significant or even measurable way.
|Comment by Andy Fingerhut [ 11/Oct/13 4:06 PM ]|
Howard, sorry I do not have more useful comments on the changes you make in your patch. Right now I only have a couple of minor comments on its form. The preferred format for patches is that created using the instructions shown on this wiki page: http://dev.clojure.org/display/community/Developing+Patches
Also, there are several parts of your patch that appear to only make changes in the whitespace of lines. It would be best to leave such changes out of a proposed patch.
|Comment by Howard Lewis Ship [ 11/Oct/13 5:00 PM ]|
Yes, I didn't notice the whitespace changes until after; I must have hit reformat at some point, despite my best efforts. I'll put together a new patch shortly.
|Comment by Howard Lewis Ship [ 11/Oct/13 6:26 PM ]|