[PATCH] [REFACTOR] Sema::MatchTemplateParametersToScopeSpecifier

[Faisal Vali]

Hi rsmith, doug.gregor,

Refactor MatchTemplateParameterToScopeSpecifier by:
  - breaking the behemoth into a bunch of smaller functions and associating them into an implementation class
  - separating some of the diagnostic details out into its own class. 

I am submitting this refactoring for review (and not committing it right away) because i'm not convinced the refactoring
is a clear improvement - specifically, I wonder if this is making the code harder to read?  Some seasoned thoughts on 
this refactoring will be much appreciated.

This refactoring is being performed in anticipation of submitting patches for DR727 which will allow explicit specializations
to be defined in-class/out-of-class.  Additionally there is a slight change in the diagnostic for PR19340 which Richard committed as: r206444.  
The message now indicates that the specialization is merely missing an empty template header (and will become legal once that is provided post DR727).

Thanks!



http://reviews.llvm.org/D3433

Files:
  lib/Sema/SemaTemplate.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3433.1.patch
Type: text/x-patch
Size: 43619 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140418/fef6243c/attachment.bin>