Skip to content

Commit

Permalink
8261606: Surprising behavior of step over in String switch
Browse files Browse the repository at this point in the history
Reviewed-by: vromero
  • Loading branch information
lahodaj committed Mar 2, 2021
1 parent be67aaa commit 85a5ae8
Show file tree
Hide file tree
Showing 5 changed files with 80 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -3877,7 +3877,9 @@ public JCTree visitStringSwitch(JCTree tree, JCExpression selector, List<JCCase>

stmtList.append(switch2);

return make.Block(0L, stmtList.toList());
JCBlock res = make.Block(0L, stmtList.toList());
res.endpos = TreeInfo.endPos(tree);
return res;
} else {
JCSwitchExpression switch2 = make.SwitchExpression(make.Ident(dollar_tmp), lb.toList());

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2821,6 +2821,7 @@ public JCStatement parseSimpleStatement() {
accept(LBRACE);
List<JCCase> cases = switchBlockStatementGroups();
JCSwitch t = to(F.at(pos).Switch(selector, cases));
t.endpos = token.endPos;
accept(RBRACE);
return t;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1246,6 +1246,8 @@ public Tag getTag() {
public static class JCSwitch extends JCStatement implements SwitchTree {
public JCExpression selector;
public List<JCCase> cases;
/** Position of closing brace, optional. */
public int endpos = Position.NOPOS;
protected JCSwitch(JCExpression selector, List<JCCase> cases) {
this.selector = selector;
this.cases = cases;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -427,6 +427,9 @@ else if (tree.hasTag(TRY)) {
JCTry t = (JCTry) tree;
return endPos((t.finalizer != null) ? t.finalizer
: (t.catchers.nonEmpty() ? t.catchers.last().body : t.body));
} else if (tree.hasTag(SWITCH) &&
((JCSwitch) tree).endpos != Position.NOPOS) {
return ((JCSwitch) tree).endpos;
} else if (tree.hasTag(SWITCH_EXPRESSION) &&
((JCSwitchExpression) tree).endpos != Position.NOPOS) {
return ((JCSwitchExpression) tree).endpos;
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,71 @@
/*
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/

/*
* @test
* @bug 8261606
* @summary Tests a line number table attribute for language constructions in different containers.
* @library /tools/lib /tools/javac/lib ../lib
* @modules jdk.compiler/com.sun.tools.javac.api
* jdk.compiler/com.sun.tools.javac.main
* jdk.compiler/com.sun.tools.javac.util
* jdk.jdeps/com.sun.tools.classfile
* @build toolbox.ToolBox InMemoryFileManager TestBase
* @build LineNumberTestBase TestCase
* @run main StringSwitchBreaks
*/

import java.util.List;

public class StringSwitchBreaks extends LineNumberTestBase {
public static void main(String[] args) throws Exception {
new StringSwitchBreaks().test();
}

public void test() throws Exception {
test(List.of(TEST_CASE));
}

private static final TestCase[] TEST_CASE = new TestCase[] {
new TestCase("""
public class StringSwitchBreaks { // 1
private void test(String s) { // 2
if (s != null) { // 3
switch (s) { // 4
case "a": // 5
System.out.println("a"); // 6
break; // 7
default: // 8
System.out.println("default"); // 9
} //10
} else { //11
System.out.println("null"); //12
} //13
} //14
} //15
""",
List.of(1, 3, 4, 6, 7, 9, 10, 12, 14),
"StringSwitchBreaks")
};

}

1 comment on commit 85a5ae8

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.