<div dir="ltr">Thanks Guys, could you do commit these patches as well, since I don't have rights for it.<div><br></div><div style>Sean, I will try to come up with an "external examples" page. Also will be asking others on the devel list for projects.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 30, 2013 at 1:41 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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="HOEnZb"><div class="h5"><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><font color="#888888"><br>
-- Sean Silva<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>