<br><div class="gmail_quote">On Thu, Jul 14, 2011 at 8:51 PM, Ville Voutilainen <span dir="ltr"><<a href="mailto:ville.voutilainen@gmail.com">ville.voutilainen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On 14 July 2011 21:42, Erik Huelsmann <<a href="mailto:ehuels@gmail.com">ehuels@gmail.com</a>> wrote:<br>
> Yesterday, we fixed #154 -- the PRINT-OBJECT regression in 0.26.0. Once<br>
> again, it turns out that our object printing and stringification routines<br>
> are confusingly named. Additionally, they are also under documented: The<br>
> writeToString() method calls toString(), but doesn't document the difference<br>
> between the two.<br>
> My idea would be to disambiguate the two functions by making writeToString()<br>
> a java-side PRINT-OBJECT, renaming it to printObject() and documenting<br>
> that's what it does. Then, probably, the lisp side function %write-to-string<br>
> should be renamed to %print-object too. Probably, more adjustments are<br>
> required throughout the code base, but those are probably at the<br>
> 'implementation detail' level.<br>
> So, what's your take on it?<br>
<br>
</div></div>By all means go for it. I've never quite understood why we have writeToString()<br>
that calls toString(). Trying to find out from the codebase is<br>
non-trivial, and even<br>
our maintenance ends up making mistakes. Any clarification is a plus.<br><br></blockquote><div><br></div><div>I have a commit sitting in my working copy to do just that (well, not the %print-object part, actually). I'll wait a bit more for others to be able to comment.</div>
<div><br></div><div>Bye,</div><div><br></div><div><br></div><div>Erik.</div><div> </div></div><br>