From 608df2d49be81aa63c8861a692f3bb0e7e57c181 Mon Sep 17 00:00:00 2001 From: Yaroslav Lepenkin Date: Tue, 28 Jun 2016 13:01:19 +0300 Subject: [PATCH] [formatter] removed useless comment --- .../actions/FormatChangedTextUtil.java | 35 ------------------- .../intellij/formatting/FormatTextRanges.java | 19 ---------- 2 files changed, 54 deletions(-) diff --git a/platform/lang-impl/src/com/intellij/codeInsight/actions/FormatChangedTextUtil.java b/platform/lang-impl/src/com/intellij/codeInsight/actions/FormatChangedTextUtil.java index 56d6588635e0..9444468f9024 100644 --- a/platform/lang-impl/src/com/intellij/codeInsight/actions/FormatChangedTextUtil.java +++ b/platform/lang-impl/src/com/intellij/codeInsight/actions/FormatChangedTextUtil.java @@ -53,12 +53,6 @@ public class FormatChangedTextUtil { return ServiceManager.getService(FormatChangedTextUtil.class); } - /** - * Allows to answer if given file has changes in comparison with VCS. - * - * @param file target file - * @return true if given file has changes; false otherwise - */ public static boolean hasChanges(@NotNull PsiFile file) { final Project project = file.getProject(); final VirtualFile virtualFile = file.getVirtualFile(); @@ -69,25 +63,10 @@ public class FormatChangedTextUtil { return false; } - /** - * Allows to answer if any file below the given directory (any level of nesting) has changes in comparison with VCS. - * - * @param directory target directory to check - * @return true if any file below the given directory has changes in comparison with VCS; - * false otherwise - */ public static boolean hasChanges(@NotNull PsiDirectory directory) { return hasChanges(directory.getVirtualFile(), directory.getProject()); } - /** - * Allows to answer if given file or any file below the given directory (any level of nesting) has changes in comparison with VCS. - * - * @param file target directory to check - * @param project target project - * @return true if given file or any file below the given directory has changes in comparison with VCS; - * false otherwise - */ public static boolean hasChanges(@NotNull VirtualFile file, @NotNull Project project) { final Collection changes = ChangeListManager.getInstance(project).getChangesIn(file); for (Change change : changes) { @@ -106,13 +85,6 @@ public class FormatChangedTextUtil { return false; } - /** - * Allows to answer if any file that belongs to the given module has changes in comparison with VCS. - * - * @param module target module to check - * @return true if any file that belongs to the given module has changes in comparison with VCS - * false otherwise - */ public static boolean hasChanges(@NotNull Module module) { final ModuleRootManager rootManager = ModuleRootManager.getInstance(module); for (VirtualFile root : rootManager.getSourceRoots()) { @@ -123,13 +95,6 @@ public class FormatChangedTextUtil { return false; } - /** - * Allows to answer if any file that belongs to the given project has changes in comparison with VCS. - * - * @param project target project to check - * @return true if any file that belongs to the given project has changes in comparison with VCS - * false otherwise - */ public static boolean hasChanges(@NotNull final Project project) { final ModifiableModuleModel moduleModel = new ReadAction() { @Override diff --git a/platform/lang-impl/src/com/intellij/formatting/FormatTextRanges.java b/platform/lang-impl/src/com/intellij/formatting/FormatTextRanges.java index 01085b32381c..3c6bcedfacaf 100644 --- a/platform/lang-impl/src/com/intellij/formatting/FormatTextRanges.java +++ b/platform/lang-impl/src/com/intellij/formatting/FormatTextRanges.java @@ -36,15 +36,6 @@ public class FormatTextRanges { myRanges.add(new FormatTextRange(range, processHeadingWhitespace)); } - /** - * Batches {@link FormatTextRange#isWhitespaceReadOnly(TextRange)} operation for all aggregated ranges. - *

- * I.e. this method allows to check if given range has intersections with any of aggregated ranges. - * - * @param range range to check - * @return true if given range doesn't have intersections with all aggregated ranges; - * false if given range intersects at least one of aggregated ranges - */ public boolean isWhitespaceReadOnly(TextRange range) { for (FormatTextRange formatTextRange : myRanges) { if (!formatTextRange.isWhitespaceReadOnly(range)) { @@ -54,16 +45,6 @@ public class FormatTextRanges { return true; } - /** - * Batches {@link FormatTextRange#isReadOnly(TextRange, boolean)} operation for all aggregated ranges. - *

- * I.e. this method allows to check if given range has intersections with any of aggregated ranges. - * - * @param range range to check - * @param rootIsRightBlock flag to use during {@link FormatTextRange#isReadOnly(TextRange, boolean)} processing - * @return true if given range doesn't have intersections with all aggregated ranges; - * false if given range intersects at least one of aggregated ranges - */ public boolean isReadOnly(TextRange range, boolean rootIsRightBlock) { for (FormatTextRange formatTextRange : myRanges) { if (!formatTextRange.isReadOnly(range, rootIsRightBlock)) {