Skip to content

Commit ee79901

Browse files
authored
[cq] Some minor Java language cleanups (#14)
1 parent c8998a3 commit ee79901

File tree

6 files changed

+11
-8
lines changed

6 files changed

+11
-8
lines changed

third_party/src/main/java/com/jetbrains/lang/dart/ide/actions/DartPubActionBase.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ abstract class DartPubActionBase : AnAction(), DumbAware {
162162
private var myProcessHandler: OSProcessHandler? = null
163163

164164
init {
165-
registerCustomShortcutSet(CommonShortcuts.getRerun(), info.console.getComponent())
165+
registerCustomShortcutSet(CommonShortcuts.getRerun(), info.console.component)
166166
}
167167

168168
fun setProcessHandler(processHandler: OSProcessHandler) {

third_party/src/main/java/com/jetbrains/lang/dart/ide/index/DartComponentInfo.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
import com.jetbrains.lang.dart.DartComponentType;
55
import org.jetbrains.annotations.Nullable;
66

7+
import java.util.Objects;
8+
79
public class DartComponentInfo {
810
private final @Nullable DartComponentType myComponentType;
911
private final @Nullable String myLibraryName;
@@ -29,7 +31,7 @@ public boolean equals(Object o) {
2931
DartComponentInfo info = (DartComponentInfo)o;
3032

3133
if (myComponentType != info.myComponentType) return false;
32-
if (myLibraryName != null ? !myLibraryName.equals(info.myLibraryName) : info.myLibraryName != null) return false;
34+
if (!Objects.equals(myLibraryName, info.myLibraryName)) return false;
3335

3436
return true;
3537
}

third_party/src/main/java/com/jetbrains/lang/dart/ide/index/DartImportOrExportInfo.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import org.jetbrains.annotations.NotNull;
55
import org.jetbrains.annotations.Nullable;
66

7+
import java.util.Objects;
78
import java.util.Set;
89

910
public class DartImportOrExportInfo implements DartShowHideInfo {
@@ -58,7 +59,7 @@ public boolean equals(Object o) {
5859

5960
if (myKind != info.myKind) return false;
6061
if (!myUri.equals(info.myUri)) return false;
61-
if (myImportPrefix != null ? !myImportPrefix.equals(info.myImportPrefix) : info.myImportPrefix != null) return false;
62+
if (!Objects.equals(myImportPrefix, info.myImportPrefix)) return false;
6263
if (!myShowComponents.equals(info.myShowComponents)) return false;
6364
if (!myHideComponents.equals(info.myHideComponents)) return false;
6465

third_party/src/main/java/com/jetbrains/lang/dart/ide/template/macro/DartClassNameMethodNameMacro.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public Result calculateResult(Expression @NotNull [] params, final ExpressionCon
1818
final Result methodNameResult = (new DartMethodNameMacro()).calculateResult(params, context);
1919

2020
if (classNameResult != null && methodNameResult != null) {
21-
return new TextResult(classNameResult.toString() + "." + methodNameResult.toString());
21+
return new TextResult(classNameResult + "." + methodNameResult);
2222
}
2323
else if (classNameResult == null && methodNameResult != null) {
2424
return new TextResult(methodNameResult.toString());

third_party/src/main/java/com/jetbrains/lang/dart/util/DartPresentableUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ else if (simpleFormalParameter != null) {
140140
}
141141
else {
142142
if (displayFinalKeyword && simpleFormalParameter.isFinal()) {
143-
result.append(DartTokenTypes.FINAL.toString());
143+
result.append(DartTokenTypes.FINAL);
144144
result.append(SPACE);
145145
}
146146
final DartType type = simpleFormalParameter.getType();
@@ -153,7 +153,7 @@ else if (simpleFormalParameter != null) {
153153
}
154154
else {
155155
if (displayFinalKeyword && simpleFormalParameter.isFinal()) {
156-
result.append(DartTokenTypes.FINAL.toString());
156+
result.append(DartTokenTypes.FINAL);
157157
result.append(SPACE);
158158
}
159159
final DartType type = simpleFormalParameter.getType();

third_party/src/main/java/com/jetbrains/lang/dart/util/DartPsiImplUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ else if (quotedDartString.startsWith(R_QUOTE)) {
106106
}
107107

108108
final DartUriElement uriElement = partOfStatement.getUriElement();
109-
assert uriElement != null : "[" + partOfStatement.toString() + "]";
109+
assert uriElement != null : "[" + partOfStatement + "]";
110110

111111
final String uri = uriElement.getUriStringAndItsRange().first;
112112
final VirtualFile file = DartResolveUtil.getRealVirtualFile(partOfStatement.getContainingFile());
@@ -121,7 +121,7 @@ else if (quotedDartString.startsWith(R_QUOTE)) {
121121
}
122122

123123
final DartUriElement uriElement = partOfStatement.getUriElement();
124-
assert uriElement != null : "[" + partOfStatement.toString() + "]";
124+
assert uriElement != null : "[" + partOfStatement + "]";
125125

126126
final String uri = uriElement.getUriStringAndItsRange().first;
127127
final VirtualFile file = DartResolveUtil.getRealVirtualFile(partOfStatement.getContainingFile());

0 commit comments

Comments
 (0)