From f963dee4e5019b6473afe8973e76e907ee31a50a Mon Sep 17 00:00:00 2001 From: Tagir Valeev Date: Thu, 12 Jan 2017 10:52:42 +0700 Subject: [PATCH] StreamApiMigrationInspection: BufferedReaderSource is considered non-trivial to mimic old behavior --- .../streamMigration/StreamApiMigrationInspection.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/java/java-impl/src/com/intellij/codeInspection/streamMigration/StreamApiMigrationInspection.java b/java/java-impl/src/com/intellij/codeInspection/streamMigration/StreamApiMigrationInspection.java index 4ba40b052dfc..a68f29c7f5d6 100644 --- a/java/java-impl/src/com/intellij/codeInspection/streamMigration/StreamApiMigrationInspection.java +++ b/java/java-impl/src/com/intellij/codeInspection/streamMigration/StreamApiMigrationInspection.java @@ -556,7 +556,12 @@ public class StreamApiMigrationInspection extends BaseJavaBatchLocalInspectionTo // Don't suggest to convert the loop which can be trivially replaced via addAll: // this is covered by UseBulkOperationInspection and ManualArrayToCollectionCopyInspection if(addAll) return null; - if(!REPLACE_TRIVIAL_FOREACH && !tb.hasOperations() && terminal.isTrivial()) return null; + if (!REPLACE_TRIVIAL_FOREACH && + !tb.hasOperations() && + !(tb.getLastOperation() instanceof BufferedReaderLines) && + terminal.isTrivial()) { + return null; + } return new CollectMigration(terminal.getMethodName()); } }