Skip to content

Commit 989b573

Browse files
committed
[CSDiag] NFC: Remove extraneous argument diagnostics which are now obsolete
1 parent 9913f8d commit 989b573

File tree

1 file changed

+0
-51
lines changed

1 file changed

+0
-51
lines changed

lib/Sema/CSDiag.cpp

Lines changed: 0 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -2156,57 +2156,6 @@ class ArgumentMatcher : public MatchCallArgumentListener {
21562156
ParamInfo(paramInfo), Arguments(args), CandidateInfo(CCI),
21572157
IsSubscript(isSubscript) {}
21582158

2159-
bool extraArgument(unsigned extraArgIdx) override {
2160-
auto name = Arguments[extraArgIdx].getLabel();
2161-
Expr *arg = ArgExpr;
2162-
2163-
auto tuple = dyn_cast<TupleExpr>(ArgExpr);
2164-
if (tuple)
2165-
arg = tuple->getElement(extraArgIdx);
2166-
2167-
auto loc = arg->getLoc();
2168-
if (tuple && extraArgIdx == tuple->getNumElements() - 1 &&
2169-
tuple->hasTrailingClosure())
2170-
TC.diagnose(loc, diag::extra_trailing_closure_in_call)
2171-
.highlight(arg->getSourceRange());
2172-
else if (Parameters.empty()) {
2173-
auto Paren = dyn_cast<ParenExpr>(ArgExpr);
2174-
Expr *SubExpr = nullptr;
2175-
if (Paren) {
2176-
SubExpr = Paren->getSubExpr();
2177-
}
2178-
2179-
if (SubExpr && CandidateInfo.CS.getType(SubExpr) &&
2180-
CandidateInfo.CS.getType(SubExpr)->isVoid()) {
2181-
TC.diagnose(loc, diag::extra_argument_to_nullary_call)
2182-
.fixItRemove(SubExpr->getSourceRange());
2183-
} else {
2184-
TC.diagnose(loc, diag::extra_argument_to_nullary_call)
2185-
.highlight(ArgExpr->getSourceRange());
2186-
}
2187-
} else if (name.empty())
2188-
TC.diagnose(loc, diag::extra_argument_positional)
2189-
.highlight(arg->getSourceRange());
2190-
else
2191-
TC.diagnose(loc, diag::extra_argument_named, name)
2192-
.highlight(arg->getSourceRange());
2193-
2194-
Diagnosed = true;
2195-
return true;
2196-
}
2197-
2198-
bool missingLabel(unsigned paramIdx) override {
2199-
return false;
2200-
}
2201-
2202-
bool extraneousLabel(unsigned paramIdx) override {
2203-
return false;
2204-
}
2205-
2206-
bool incorrectLabel(unsigned paramIdx) override {
2207-
return false;
2208-
}
2209-
22102159
bool outOfOrderArgument(unsigned argIdx, unsigned prevArgIdx) override {
22112160
auto &cs = CandidateInfo.CS;
22122161
OutOfOrderArgumentFailure failure(nullptr, cs, argIdx, prevArgIdx, Bindings,

0 commit comments

Comments
 (0)