<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 5, 2014 at 9:50 PM, Nikola Smiljanić <span dir="ltr"><<a href="mailto:popizdeh@gmail.com" target="_blank">popizdeh@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Committed these tests in r210306 but I'll leave this review request open in case the following code produces something in the next week or so :)<br>

<br>
  richard_smith_engine e(beer()); // not sure if this is a proper way to seed the engine<br></blockquote><div><br></div><div>Sorry, that's a function declaration... =)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  obscure_frienddecl_distribution d(e);<br>
  d();</blockquote><div><br></div><div>Should friendDecl also match friend template declarations? You don't have tests for that case. Also, there's no test coverage here for the case where the 'friend' keyword is not the first token (nor for the case of a friend function defined inline in the class):</div>
<div><br></div><div>    int inline friend f() { return 0; }</div></div></div></div>