Skip to content

Commit 9bc288c

Browse files
committed
[INTERNAL] additional code cleanup
1 parent 23c2052 commit 9bc288c

File tree

3 files changed

+6
-12
lines changed

3 files changed

+6
-12
lines changed

src/main/java/net/seesharpsoft/intellij/plugins/csv/CsvLexer.flex

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import com.intellij.lexer.FlexLexer;
44
import com.intellij.psi.tree.IElementType;
55
import com.intellij.openapi.project.Project;
66
import net.seesharpsoft.intellij.plugins.csv.psi.CsvTypes;
7-
import static net.seesharpsoft.intellij.plugins.csv.settings.CsvCodeStyleSettings.getCurrentSeparator;
87
import com.intellij.psi.TokenType;
98

109
%%

src/main/java/net/seesharpsoft/intellij/plugins/csv/intention/CsvShiftColumnIntentionAction.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ protected CsvShiftColumnIntentionAction(String text) {
1818
super(text);
1919
}
2020

21-
protected static void changeLeftAndRightColumnOrder(@NotNull Project project, CsvFile psiFile, CsvColumnInfo<PsiElement> leftColumnInfo, CsvColumnInfo<PsiElement> rightColumnInfo) {
22-
Document document = PsiDocumentManager.getInstance(project).getDocument(psiFile);
23-
document.setText(changeLeftAndRightColumnOrder(document.getText(), CsvCodeStyleSettings.getCurrentSeparator(project, psiFile.getLanguage()), leftColumnInfo, rightColumnInfo));
21+
protected static void changeLeftAndRightColumnOrder(@NotNull Project project, CsvFile csvFile, CsvColumnInfo<PsiElement> leftColumnInfo, CsvColumnInfo<PsiElement> rightColumnInfo) {
22+
Document document = PsiDocumentManager.getInstance(project).getDocument(csvFile);
23+
document.setText(changeLeftAndRightColumnOrder(document.getText(), CsvCodeStyleSettings.getCurrentSeparator(project, csvFile.getLanguage()), leftColumnInfo, rightColumnInfo));
2424
}
2525

2626
@NotNull
@@ -43,10 +43,10 @@ protected static String changeLeftAndRightColumnOrder(String text, String separa
4343
TextRange.create(middleSeparator.getEndOffset(), middleSeparator.getEndOffset()) :
4444
findNextSeparatorOrCRLF(rightElement);
4545

46-
newText.append(text.substring(lastIndex, leftSeparator.getEndOffset()))
47-
.append(text.substring(middleSeparator.getEndOffset(), rightSeparator.getStartOffset()))
46+
newText.append(text, lastIndex, leftSeparator.getEndOffset())
47+
.append(text, middleSeparator.getEndOffset(), rightSeparator.getStartOffset())
4848
.append(separator)
49-
.append(text.substring(leftSeparator.getEndOffset(), middleSeparator.getStartOffset()));
49+
.append(text, leftSeparator.getEndOffset(), middleSeparator.getStartOffset());
5050

5151
lastIndex = rightSeparator.getStartOffset();
5252
}

src/main/java/net/seesharpsoft/intellij/plugins/csv/settings/CsvCodeStyleSettings.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import com.intellij.lang.Language;
44
import com.intellij.openapi.application.ApplicationManager;
55
import com.intellij.openapi.project.Project;
6-
import com.intellij.openapi.project.ProjectCoreUtil;
76
import com.intellij.psi.codeStyle.CodeStyleSettings;
87
import com.intellij.psi.codeStyle.CodeStyleSettingsManager;
98
import com.intellij.psi.codeStyle.CustomCodeStyleSettings;
@@ -48,10 +47,6 @@ public static String getCurrentSeparator(@Nullable Project project, @Nullable La
4847
return getCurrentSeparator(project);
4948
}
5049

51-
public static String getCurrentSeparator() {
52-
return getCurrentSeparator(ProjectCoreUtil.theOnlyOpenProject());
53-
}
54-
5550
public CsvCodeStyleSettings(CodeStyleSettings settings) {
5651
super("CsvCodeStyleSettings", settings);
5752
}

0 commit comments

Comments
 (0)