Skip to content

Commit 0d2f585

Browse files
committed
Merge pull request swiftlang#1453 from practicalswift/remove-unused-argument
[AST] Remove unused maybeAddSameTypeRequirementForNestedType(…)-argument
2 parents 10e0151 + 97586ee commit 0d2f585

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

lib/AST/ArchetypeBuilder.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,6 @@ static ProtocolConformance *getSuperConformance(
299299
/// If there is a same-type requirement to be added for the given nested type
300300
/// due to a superclass constraint on the parent type, add it now.
301301
static void maybeAddSameTypeRequirementForNestedType(
302-
ArchetypeBuilder::PotentialArchetype *parentPA,
303302
ArchetypeBuilder::PotentialArchetype *nestedPA,
304303
RequirementSource fromSource,
305304
ProtocolConformance *superConformance,
@@ -369,7 +368,7 @@ bool ArchetypeBuilder::PotentialArchetype::addConformance(
369368

370369
// If there's a superclass constraint that conforms to the protocol,
371370
// add the appropriate same-type relationship.
372-
maybeAddSameTypeRequirementForNestedType(this, known->second.front(),
371+
maybeAddSameTypeRequirementForNestedType(known->second.front(),
373372
source, superConformance,
374373
builder);
375374
continue;
@@ -387,7 +386,7 @@ bool ArchetypeBuilder::PotentialArchetype::addConformance(
387386

388387
// If there's a superclass constraint that conforms to the protocol,
389388
// add the appropriate same-type relationship.
390-
maybeAddSameTypeRequirementForNestedType(this, otherPA, source,
389+
maybeAddSameTypeRequirementForNestedType(otherPA, source,
391390
superConformance, builder);
392391
}
393392

@@ -491,7 +490,7 @@ auto ArchetypeBuilder::PotentialArchetype::getNestedType(
491490
// add the appropriate same-type relationship.
492491
ProtocolConformance *superConformance =
493492
getSuperConformance(this, conforms.first, conforms.second, builder);
494-
maybeAddSameTypeRequirementForNestedType(this, pa, source,
493+
maybeAddSameTypeRequirementForNestedType(pa, source,
495494
superConformance, builder);
496495
}
497496
}
@@ -969,7 +968,7 @@ bool ArchetypeBuilder::addSuperclassRequirement(PotentialArchetype *T,
969968

970969
for (auto nestedPA : nested->second) {
971970
if (nestedPA->getResolvedAssociatedType() == assocType)
972-
maybeAddSameTypeRequirementForNestedType(T, nestedPA,
971+
maybeAddSameTypeRequirementForNestedType(nestedPA,
973972
Source,
974973
superConformance, *this);
975974
}

0 commit comments

Comments
 (0)