From 64efa8e19172a39cdf757244cf763b8a8a8375eb Mon Sep 17 00:00:00 2001 From: "Roman.Ivanov" Date: Wed, 3 Apr 2019 15:12:06 +0700 Subject: [PATCH] JoiningMigration: remove unnecessary side effect check for prefix --- .../codeInspection/streamMigration/JoiningMigration.java | 1 - 1 file changed, 1 deletion(-) diff --git a/java/java-impl/src/com/intellij/codeInspection/streamMigration/JoiningMigration.java b/java/java-impl/src/com/intellij/codeInspection/streamMigration/JoiningMigration.java index 9a0880839ba2..c7f01498b918 100644 --- a/java/java-impl/src/com/intellij/codeInspection/streamMigration/JoiningMigration.java +++ b/java/java-impl/src/com/intellij/codeInspection/streamMigration/JoiningMigration.java @@ -728,7 +728,6 @@ List builderStrInitializers = null; if(builderStrInitializers != null) { prefixJoinParts.addAll(0, builderStrInitializers); } - if (prefixJoinParts.stream().anyMatch(joinPart -> SideEffectChecker.mayHaveSideEffects(joinPart))) return null; if (afterLoopAppend != null && VariableAccessUtils.variableIsUsed(targetBuilder, afterLoopAppend.getArgumentList())) return null; List suffixJoinParts = extractJoinParts(afterLoopAppend); if (suffixJoinParts == null) return null;