Skip to content

Commit 0dba5f7

Browse files
committed
Use return values in OverloadResolution, drop Error Prone opt-out
1 parent 7f329f4 commit 0dba5f7

3 files changed

Lines changed: 19 additions & 22 deletions

File tree

tests/minimized/BUILD

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,5 @@ java_library(
1111
],
1212
javacopts = [
1313
"'-Xplugin:scip -build-tool:bazel'",
14-
# These are deliberately minimal SCIP indexer inputs (e.g. ignored
15-
# return values), not production code, so skip Error Prone linting.
16-
"-XepDisableAllChecks",
1714
],
1815
)

tests/minimized/src/main/java/minimized/OverloadResolution.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66
* symbols (lastIndexOf(), (+1), (+2), (+3)).
77
*/
88
public class OverloadResolution {
9-
public static void app() {
10-
"".lastIndexOf('a');
11-
"".lastIndexOf('b', 0);
12-
"".lastIndexOf("c");
13-
"".lastIndexOf("d", 0);
9+
public static int app() {
10+
return "".lastIndexOf('a')
11+
+ "".lastIndexOf('b', 0)
12+
+ "".lastIndexOf("c")
13+
+ "".lastIndexOf("d", 0);
1414
}
1515
}

tests/snapshots/src/main/generated/tests/minimized/src/main/java/minimized/OverloadResolution.java

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -22,20 +22,20 @@ public class OverloadResolution {
2222
// signature_documentation
2323
// > public OverloadResolution()
2424
// ⌄ enclosing_range_start scip-java maven . . minimized/OverloadResolution#app().
25-
public static void app() {
26-
// ^^^ definition scip-java maven . . minimized/OverloadResolution#app().
27-
// kind StaticMethod
28-
// display_name app
29-
// signature_documentation
30-
// > public static void app()
31-
"".lastIndexOf('a');
32-
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf().
33-
"".lastIndexOf('b', 0);
34-
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+1).
35-
"".lastIndexOf("c");
36-
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+2).
37-
"".lastIndexOf("d", 0);
38-
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+3).
25+
public static int app() {
26+
// ^^^ definition scip-java maven . . minimized/OverloadResolution#app().
27+
// kind StaticMethod
28+
// display_name app
29+
// signature_documentation
30+
// > public static int app()
31+
return "".lastIndexOf('a')
32+
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf().
33+
+ "".lastIndexOf('b', 0)
34+
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+1).
35+
+ "".lastIndexOf("c")
36+
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+2).
37+
+ "".lastIndexOf("d", 0);
38+
// ^^^^^^^^^^^ reference scip-java maven jdk 11 java/lang/String#lastIndexOf(+3).
3939
}
4040
// ⌃ enclosing_range_end scip-java maven . . minimized/OverloadResolution#app().
4141
}

0 commit comments

Comments
 (0)