<div dir="ltr">The last two sound better than the others to me, FWIW.<div><br></div><div>Robby</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 2, 2013 at 2:37 PM, Ryan Culpepper <span dir="ltr"><<a href="mailto:ryanc@ccs.neu.edu" target="_blank">ryanc@ccs.neu.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 10/02/2013 03:19 PM, Stephen Chang wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can I push the attached (1-line) patch? I don't have a good grasp of<br>
namespaces so I would like someone to review it first.<br>
<br>
Right now, make-base-eval tries to attach racket/pretty to the<br>
namespace regardless of whether it's already there, which sometimes<br>
results in an exception (for example if the #lang given to<br>
make-base-eval is racket).<br>
<br>
The patch simply first checks if the desired identifier is already there.<br>
</blockquote>
<br></div></div>
Whether that identifier exists in the namespace has nothing to do with whether racket/pretty can be attached.<br>
<br>
One option would be for install-pretty-printer! to just catch and discard the error. Evaluators for some languages would mysteriously not have pretty-printing turned on by default.<br>
<br>
Another option would be to attach racket/pretty before requiring the initial language for the namespace.<br>
<br>
Another option is use #:pretty-print? #f when attaching racket/pretty would fail.<br>
<br>
Ryan<br>
<br>
_________________________<br>
 Racket Developers list:<br>
 <a href="http://lists.racket-lang.org/dev" target="_blank">http://lists.racket-lang.org/<u></u>dev</a><br>
</blockquote></div><br></div>