Skip to content

Commit f3a80ea

Browse files
committed
NFC: Move some code around
1 parent caeab58 commit f3a80ea

File tree

2 files changed

+29
-32
lines changed

2 files changed

+29
-32
lines changed

tools/SourceKit/lib/SwiftLang/SwiftEditor.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2413,9 +2413,9 @@ void SwiftLangSupport::editorReplaceText(StringRef Name,
24132413
return;
24142414
}
24152415

2416-
// If client doesn't need any information, we doesn't need to parse it.
24172416
EditorDoc->resetSyntaxInfo(Snapshot, *this);
24182417

2418+
// If client doesn't need any information, we doesn't need to parse it.
24192419
if (!Consumer.documentStructureEnabled() &&
24202420
!Consumer.syntaxMapEnabled() &&
24212421
!Consumer.diagnosticsEnabled()) {

tools/SourceKit/tools/sourcekitd/lib/Service/Requests.cpp

Lines changed: 28 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -3514,37 +3514,6 @@ void SKEditorConsumer::recordFormattedText(StringRef Text) {
35143514
Dict.set(KeySourceText, Text);
35153515
}
35163516

3517-
static void fillDictionaryForDiagnosticInfoBase(
3518-
ResponseBuilder::Dictionary Elem, const DiagnosticEntryInfoBase &Info);
3519-
3520-
static void fillDictionaryForDiagnosticInfo(
3521-
ResponseBuilder::Dictionary Elem, const DiagnosticEntryInfo &Info) {
3522-
3523-
UIdent SeverityUID;
3524-
static UIdent UIDKindDiagWarning(KindDiagWarning.str());
3525-
static UIdent UIDKindDiagError(KindDiagError.str());
3526-
switch (Info.Severity) {
3527-
case DiagnosticSeverityKind::Warning:
3528-
SeverityUID = UIDKindDiagWarning;
3529-
break;
3530-
case DiagnosticSeverityKind::Error:
3531-
SeverityUID = UIDKindDiagError;
3532-
break;
3533-
}
3534-
3535-
Elem.set(KeySeverity, SeverityUID);
3536-
fillDictionaryForDiagnosticInfoBase(Elem, Info);
3537-
3538-
if (!Info.Notes.empty()) {
3539-
auto NotesArr = Elem.setArray(KeyDiagnostics);
3540-
for (auto &NoteDiag : Info.Notes) {
3541-
auto NoteElem = NotesArr.appendDictionary();
3542-
NoteElem.set(KeySeverity, KindDiagNote);
3543-
fillDictionaryForDiagnosticInfoBase(NoteElem, NoteDiag);
3544-
}
3545-
}
3546-
}
3547-
35483517
static void fillDictionaryForDiagnosticInfoBase(
35493518
ResponseBuilder::Dictionary Elem, const DiagnosticEntryInfoBase &Info) {
35503519

@@ -3603,6 +3572,34 @@ static void fillDictionaryForDiagnosticInfoBase(
36033572
}
36043573
}
36053574

3575+
static void fillDictionaryForDiagnosticInfo(
3576+
ResponseBuilder::Dictionary Elem, const DiagnosticEntryInfo &Info) {
3577+
3578+
UIdent SeverityUID;
3579+
static UIdent UIDKindDiagWarning(KindDiagWarning.str());
3580+
static UIdent UIDKindDiagError(KindDiagError.str());
3581+
switch (Info.Severity) {
3582+
case DiagnosticSeverityKind::Warning:
3583+
SeverityUID = UIDKindDiagWarning;
3584+
break;
3585+
case DiagnosticSeverityKind::Error:
3586+
SeverityUID = UIDKindDiagError;
3587+
break;
3588+
}
3589+
3590+
Elem.set(KeySeverity, SeverityUID);
3591+
fillDictionaryForDiagnosticInfoBase(Elem, Info);
3592+
3593+
if (!Info.Notes.empty()) {
3594+
auto NotesArr = Elem.setArray(KeyDiagnostics);
3595+
for (auto &NoteDiag : Info.Notes) {
3596+
auto NoteElem = NotesArr.appendDictionary();
3597+
NoteElem.set(KeySeverity, KindDiagNote);
3598+
fillDictionaryForDiagnosticInfoBase(NoteElem, NoteDiag);
3599+
}
3600+
}
3601+
}
3602+
36063603
void SKEditorConsumer::setDiagnosticStage(UIdent DiagStage) {
36073604
Dict.set(KeyDiagnosticStage, DiagStage);
36083605
}

0 commit comments

Comments
 (0)