On Thu, Jun 5, 2014 at 9:50 PM, Nikola Smiljanić <[email protected]> wrote:

> 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 :)
>
>   richard_smith_engine e(beer()); // not sure if this is a proper way to
> seed the engine
>

Sorry, that's a function declaration... =)


>   obscure_frienddecl_distribution d(e);
>   d();


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):

    int inline friend f() { return 0; }
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to