<div dir="ltr">LGTM for the escaping example fix (Sean: we currently don't support single-tick unescaping - and so far nobody has asked for it :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 30, 2013 at 12:27 AM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">klimek, could you review this escaping patch? The current escaping<br>
there doesn't look broken (i.e. it looks like valid JSON).<br>
<br>
The second patch (about BEAR) LGTM.<br>
<br>
btw, I encourage you to share any cool stuff you do with Clang (just<br>
saw that you have more stuff on github). I would like to start an<br>
"external examples" page; if you send a patch starting such a new page<br>
I would be elated (you can use<br>
<<a href="http://llvm.org/docs/SphinxQuickstartTemplate.html" target="_blank">http://llvm.org/docs/SphinxQuickstartTemplate.html</a>> as a basis for<br>
the new page).<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Sean Silva<br>
</font></span></blockquote></div><br></div>