Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSergey Prigogin2013-09-11 03:10:16 +0000
committerSergey Prigogin2013-09-11 03:25:28 +0000
commit7372e983a16e7593835f108d1544e625e4e223b8 (patch)
tree34acb307c79ca6e0afb5575b1cddae18a0e4b0a4
parent22078c723dc15fe30a8111b0c833cee1422e5bc9 (diff)
downloadorg.eclipse.cdt-7372e983a16e7593835f108d1544e625e4e223b8.tar.gz
org.eclipse.cdt-7372e983a16e7593835f108d1544e625e4e223b8.tar.xz
org.eclipse.cdt-7372e983a16e7593835f108d1544e625e4e223b8.zip
Bug 416280 - Name resolution problem with alias template. Fix for the
test case in comment #6.
-rw-r--r--core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2TemplateTests.java18
-rw-r--r--core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/semantics/CPPTemplates.java33
2 files changed, 43 insertions, 8 deletions
diff --git a/core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2TemplateTests.java b/core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2TemplateTests.java
index d0823a45939..d76d0ef7913 100644
--- a/core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2TemplateTests.java
+++ b/core/org.eclipse.cdt.core.tests/parser/org/eclipse/cdt/core/parser/tests/ast2/AST2TemplateTests.java
@@ -7094,7 +7094,23 @@ public class AST2TemplateTests extends AST2TestBase {
// void test(A<int>* c) {
// f(c);
// }
- public void testAliasTemplate_416280() throws Exception {
+ public void testAliasTemplate_416280_1() throws Exception {
+ parseAndCheckBindings();
+ }
+
+ // template<typename T>
+ // struct C {};
+ //
+ // template<typename U>
+ // struct A {
+ // template<typename V>
+ // using B = C<V>;
+ // };
+ //
+ // struct D : public A<char> {
+ // B<int> b;
+ // };
+ public void testAliasTemplate_416280_2() throws Exception {
parseAndCheckBindings();
}
diff --git a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/semantics/CPPTemplates.java b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/semantics/CPPTemplates.java
index 3806b7a1b41..dfa64813752 100644
--- a/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/semantics/CPPTemplates.java
+++ b/core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/semantics/CPPTemplates.java
@@ -70,6 +70,7 @@ import org.eclipse.cdt.core.dom.ast.cpp.ICPPASTTemplateParameter;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPASTTemplateSpecialization;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPASTTemplatedTypeTemplateParameter;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPAliasTemplate;
+import org.eclipse.cdt.core.dom.ast.cpp.ICPPAliasTemplateInstance;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPClassScope;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPClassSpecialization;
import org.eclipse.cdt.core.dom.ast.cpp.ICPPClassTemplate;
@@ -686,19 +687,27 @@ public class CPPTemplates {
IASTName templateName = id.getTemplateName();
IBinding template = templateName.resolvePreBinding();
- // Alias Template.
+ // Alias template.
if (template instanceof ICPPAliasTemplate) {
ICPPAliasTemplate aliasTemplate = (ICPPAliasTemplate) template;
- IType aliasedType = aliasTemplate.getType();
ICPPTemplateArgument[] args = createTemplateArgumentArray(id);
args = addDefaultArguments(aliasTemplate, args, id);
ICPPTemplateParameterMap parameterMap = createParameterMap(aliasTemplate, args);
+ IType aliasedType = aliasTemplate.getType();
IBinding owner = template.getOwner();
- ICPPClassSpecialization within = getSpecializationContext(owner);
- IType instantiatedType = instantiateType(aliasedType, parameterMap, -1, within, id);
- StringBuilder buf= new StringBuilder();
- buf.append(id.getSimpleID()).append(ASTTypeUtil.getArgumentListString(args, false));
- return new CPPAliasTemplateInstance(buf.toString().toCharArray(), aliasTemplate, instantiatedType);
+ return createAliasTemplaceInstance(aliasTemplate, args, parameterMap, aliasedType, owner, id);
+ }
+
+ // Alias template instance.
+ if (template instanceof ICPPAliasTemplateInstance) {
+ ICPPAliasTemplateInstance aliasTemplateInstance = (ICPPAliasTemplateInstance) template;
+ ICPPTemplateArgument[] args = createTemplateArgumentArray(id);
+ ICPPAliasTemplate aliasTemplate = aliasTemplateInstance.getTemplateDefinition();
+ args = addDefaultArguments(aliasTemplate, args, id);
+ ICPPTemplateParameterMap parameterMap = createParameterMap(aliasTemplate, args);
+ IType aliasedType = aliasTemplateInstance.getType();
+ IBinding owner = aliasTemplateInstance.getOwner();
+ return createAliasTemplaceInstance(aliasTemplate, args, parameterMap, aliasedType, owner, id);
}
// Class template.
@@ -759,6 +768,16 @@ public class CPPTemplates {
}
}
+ private static IBinding createAliasTemplaceInstance(ICPPAliasTemplate aliasTemplate,
+ ICPPTemplateArgument[] args, ICPPTemplateParameterMap parameterMap, IType aliasedType,
+ IBinding owner, ICPPASTTemplateId id) {
+ ICPPClassSpecialization within = getSpecializationContext(owner);
+ IType instantiatedType = instantiateType(aliasedType, parameterMap, -1, within, id);
+ StringBuilder buf= new StringBuilder();
+ buf.append(id.getSimpleID()).append(ASTTypeUtil.getArgumentListString(args, false));
+ return new CPPAliasTemplateInstance(buf.toString().toCharArray(), aliasTemplate, instantiatedType);
+ }
+
static boolean isClassTemplate(ICPPASTTemplateId id) {
IASTNode parentOfName = id.getParent();

Back to the top