Skip to content

Commit 938b1d7

Browse files
committed
take move-switch-default-last out of default codemods set
1 parent bd7aaed commit 938b1d7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

core-codemods/src/main/java/io/codemodder/codemods/DefaultCodemods.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ public static List<Class<? extends CodeChanger>> asList() {
2929
JSPScriptletXSSCodemod.class,
3030
LimitReadlineCodemod.class,
3131
MavenSecureURLCodemod.class,
32-
MoveSwitchDefaultCaseLastCodemod.class,
3332
OutputResourceLeakCodemod.class,
3433
PreventFileWriterLeakWithFilesCodemod.class,
3534
RandomizeSeedCodemod.class,

core-codemods/src/main/java/io/codemodder/codemods/MoveSwitchDefaultCaseLastCodemod.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@
1111

1212
/**
1313
* A codemod for moving the "default" case to last in switch statements. This codemod is not
14-
* currently in the default set because it
14+
* currently in the default set because it could conceivably change behavior when other case
15+
* statements fall through to it. It should be improved to only move if the previous case
16+
* does not fall through.
1517
*/
1618
@Codemod(
1719
id = "pixee:java/move-switch-default-last",

0 commit comments

Comments
 (0)