<div dir="ltr"><div>Thanks for the feedback Manuel,<br><br></div><div>I didn't try it in MSVC 2012, I hope it's not a hassle to compile clang in Windows,<br></div><div>I'm downloading the express edition so I can try it.<br><br></div><div>I believe the sole critical piece regarding the style guide is the declaration of the<br><span class="">ParseExpressionList function. Although I think it's good</span> to have the default argument<br>initialized that way, which is simple and short, it may not be in accordance with the<br>"Do not use Braced Initializer Lists to Call a Constructor" section of the style guide.<br></div><div><br>As I'm unsure about this specific usage, I'll left it as is until proper review.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-01-13 17:12 GMT-02:00 Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">General high level comment: you're using C++11 extensively, have you made sure you're only using features MSVC 2012 supports (as per the style guide: <a href="http://llvm.org/docs/CodingStandards.html" target="_blank">http://llvm.org/docs/CodingStandards.html</a>)?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D6880" target="_blank">http://reviews.llvm.org/D6880</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div>