Skip to content

Commit 588d87b

Browse files
committed
[CodeCompletion] Remove completeCaseStmtDotPrefix() completion
Now that, normal dot-member completion works at case position. We don't need to handle it specially. rdar://problem/45219937
1 parent ce5b3cc commit 588d87b

File tree

5 files changed

+0
-35
lines changed

5 files changed

+0
-35
lines changed

include/swift/IDE/CodeCompletion.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,6 @@ enum class CompletionKind {
495495
TypeIdentifierWithoutDot,
496496
CaseStmtKeyword,
497497
CaseStmtBeginning,
498-
CaseStmtDotPrefix,
499498
NominalMemberBeginning,
500499
AccessorBeginning,
501500
AttributeBegin,

include/swift/Parse/CodeCompletionCallbacks.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -177,9 +177,6 @@ class CodeCompletionCallbacks {
177177
/// Complete at the beginning of a case stmt pattern.
178178
virtual void completeCaseStmtBeginning() {};
179179

180-
/// Complete a case stmt pattern that starts with a dot.
181-
virtual void completeCaseStmtDotPrefix() {};
182-
183180
/// Complete at the beginning of member of a nominal decl member -- no tokens
184181
/// provided by user.
185182
virtual void completeNominalMemberBeginning(

lib/IDE/CodeCompletion.cpp

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1378,7 +1378,6 @@ class CodeCompletionCallbacksImpl : public CodeCompletionCallbacks {
13781378

13791379
void completeCaseStmtKeyword() override;
13801380
void completeCaseStmtBeginning() override;
1381-
void completeCaseStmtDotPrefix() override;
13821381
void completeDeclAttrBeginning(bool Sil, bool isIndependent) override;
13831382
void completeDeclAttrParam(DeclAttrKind DK, int Index) override;
13841383
void completeInPrecedenceGroup(SyntaxKind SK) override;
@@ -4727,13 +4726,6 @@ void CodeCompletionCallbacksImpl::completeCaseStmtBeginning() {
47274726
CurDeclContext = P.CurDeclContext;
47284727
}
47294728

4730-
void CodeCompletionCallbacksImpl::completeCaseStmtDotPrefix() {
4731-
assert(!InEnumElementRawValue);
4732-
4733-
Kind = CompletionKind::CaseStmtDotPrefix;
4734-
CurDeclContext = P.CurDeclContext;
4735-
}
4736-
47374729
void CodeCompletionCallbacksImpl::completeImportDecl(
47384730
std::vector<std::pair<Identifier, SourceLoc>> &Path) {
47394731
Kind = CompletionKind::Import;
@@ -5017,7 +5009,6 @@ void CodeCompletionCallbacksImpl::addKeywords(CodeCompletionResultSink &Sink,
50175009
case CompletionKind::SuperExpr:
50185010
case CompletionKind::SuperExprDot:
50195011
case CompletionKind::CaseStmtBeginning:
5020-
case CompletionKind::CaseStmtDotPrefix:
50215012
case CompletionKind::TypeIdentifierWithDot:
50225013
case CompletionKind::TypeIdentifierWithoutDot:
50235014
break;
@@ -5423,13 +5414,6 @@ void CodeCompletionCallbacksImpl::doneParsing() {
54235414
break;
54245415
}
54255416

5426-
case CompletionKind::CaseStmtDotPrefix: {
5427-
Lookup.setHaveDot(SourceLoc());
5428-
SourceLoc Loc = P.Context.SourceMgr.getCodeCompletionLoc();
5429-
Lookup.getTypeContextEnumElementCompletions(Loc);
5430-
break;
5431-
}
5432-
54335417
case CompletionKind::NominalMemberBeginning: {
54345418
CompletionOverrideLookup OverrideLookup(CompletionContext.getResultSink(),
54355419
P.Context, CurDeclContext,

lib/IDE/TypeContextInfo.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ class ContextInfoCallbacks : public CodeCompletionCallbacks {
4646

4747
void completeUnresolvedMember(CodeCompletionExpr *E,
4848
SourceLoc DotLoc) override;
49-
void completeCaseStmtDotPrefix() override;
5049

5150
void doneParsing() override;
5251
};
@@ -84,9 +83,6 @@ void ContextInfoCallbacks::completeUnresolvedMember(CodeCompletionExpr *E,
8483
void ContextInfoCallbacks::completeCaseStmtBeginning() {
8584
// TODO: Implement?
8685
}
87-
void ContextInfoCallbacks::completeCaseStmtDotPrefix() {
88-
// TODO: Implement?
89-
}
9086

9187
void ContextInfoCallbacks::doneParsing() {
9288
if (!ParsedExpr)

lib/Parse/ParseStmt.cpp

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1096,17 +1096,6 @@ static void parseGuardedPattern(Parser &P, GuardedPattern &result,
10961096
status.setHasCodeCompletion();
10971097
return;
10981098
}
1099-
if (parsingContext == GuardedPatternContext::Case &&
1100-
P.Tok.isAny(tok::period_prefix, tok::period) &&
1101-
P.peekToken().is(tok::code_complete)) {
1102-
P.consumeToken();
1103-
if (P.CodeCompletion)
1104-
P.CodeCompletion->completeCaseStmtDotPrefix();
1105-
auto loc = P.consumeToken(tok::code_complete);
1106-
result.ThePattern = new (P.Context) AnyPattern(loc);
1107-
status.setHasCodeCompletion();
1108-
return;
1109-
}
11101099

11111100
// If this is a 'catch' clause and we have "catch {" or "catch where...",
11121101
// then we get an implicit "let error" pattern.

0 commit comments

Comments
 (0)