On 02.04.2012, at 21:15, David Blaikie wrote:

> Author: dblaikie
> Date: Mon Apr  2 14:15:28 2012
> New Revision: 153887
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=153887&view=rev
> Log:
> Correct error recovery when missing 'class' in a template template parameter.
> 
> The diagnostic message correctly informs the user that they have omitted the
> 'class' keyword, but neither suggests this insertion as a fixit, nor attempts
> to recover as if they had provided the keyword.
> 
> This fixes the recovery, adds the fixit, and adds a separate diagnostic and
> corresponding replacement fixit for cases where the user wrote 'struct' or
> 'typename' instead of 'class' (suggested by Richard Smith as a possible common
> mistake).
> 
> I'm not sure the diagnostic message for either the original or new cases feel
> very Clang-esque, so I'm open to suggestions there. The fixit hints make it
> fairly easy to see what's required, though.

"Template template parameters require 'class' after the argument list."?

Sebastian
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to