From d5d11aa5398517152d494993906121ec5add307c Mon Sep 17 00:00:00 2001 From: Eugene Zhuravlev Date: Thu, 12 Feb 2015 20:28:56 +0100 Subject: [PATCH] avoid redundant isDirectory() call; javadoc --- .../com/intellij/openapi/util/io/FileUtilRt.java | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/platform/util-rt/src/com/intellij/openapi/util/io/FileUtilRt.java b/platform/util-rt/src/com/intellij/openapi/util/io/FileUtilRt.java index 8a5be88690e1..e24561626b8d 100644 --- a/platform/util-rt/src/com/intellij/openapi/util/io/FileUtilRt.java +++ b/platform/util-rt/src/com/intellij/openapi/util/io/FileUtilRt.java @@ -532,13 +532,16 @@ public class FileUtilRt { } } + /** + * Warning! this method is _not_ symlinks-aware. Consider using com.intellij.openapi.util.io.FileUtil.delete() + * @param file file or directory to delete + * @return true if the file did not exist or was successfully deleted + */ public static boolean delete(@NotNull File file) { - if (file.isDirectory()) { - File[] files = file.listFiles(); - if (files != null) { - for (File child : files) { - if (!delete(child)) return false; - } + File[] files = file.listFiles(); + if (files != null) { + for (File child : files) { + if (!delete(child)) return false; } }