From 617d09cfe6a8113491d4eecd383342ab1f87e2da Mon Sep 17 00:00:00 2001 From: Roman Shevchenko Date: Fri, 14 Sep 2012 15:19:30 +0400 Subject: [PATCH 1/5] Typo --- .../src/com/intellij/openapi/util/DimensionService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/platform/platform-api/src/com/intellij/openapi/util/DimensionService.java b/platform/platform-api/src/com/intellij/openapi/util/DimensionService.java index 827e73cebea3..0fc4f7d49f52 100644 --- a/platform/platform-api/src/com/intellij/openapi/util/DimensionService.java +++ b/platform/platform-api/src/com/intellij/openapi/util/DimensionService.java @@ -77,7 +77,7 @@ public class DimensionService implements PersistentStateComponent, Appl * @return point stored under the specified key. The method returns * null if there is no stored value under the key. If point * is outside of current screen bounds then the method returns null. It - * properly works in multimonitor configuration. + * properly works in multi-monitor configuration. * @exception java.lang.IllegalArgumentException if key is null. * @param key a String key to perform a query for. */ From c96d980fa806b137c8e50d6cb6776c6fa3e7209e Mon Sep 17 00:00:00 2001 From: Roman Shevchenko Date: Fri, 14 Sep 2012 17:44:20 +0400 Subject: [PATCH 2/5] Logging --- .../intellij/openapi/vfs/impl/jar/JarHandler.java | 13 ++++++++----- .../src/messages/VfsBundle.properties | 2 +- 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/platform/platform-impl/src/com/intellij/openapi/vfs/impl/jar/JarHandler.java b/platform/platform-impl/src/com/intellij/openapi/vfs/impl/jar/JarHandler.java index 9a31eb444904..d2553126eee1 100644 --- a/platform/platform-impl/src/com/intellij/openapi/vfs/impl/jar/JarHandler.java +++ b/platform/platform-impl/src/com/intellij/openapi/vfs/impl/jar/JarHandler.java @@ -22,6 +22,7 @@ package com.intellij.openapi.vfs.impl.jar; import com.intellij.openapi.application.ApplicationManager; import com.intellij.openapi.application.ModalityState; import com.intellij.openapi.application.PathManager; +import com.intellij.openapi.diagnostic.Logger; import com.intellij.openapi.progress.ProgressIndicator; import com.intellij.openapi.progress.ProgressManager; import com.intellij.openapi.ui.Messages; @@ -43,6 +44,8 @@ import java.io.IOException; import java.io.OutputStream; public class JarHandler extends JarHandlerBase implements FileSystemInterface { + private static final Logger LOG = Logger.getInstance("#com.intellij.openapi.vfs.impl.jar.JarHandler"); + @NonNls private static final String JARS_FOLDER = "jars"; private final JarFileSystemImpl myFileSystem; @@ -112,17 +115,17 @@ public class JarHandler extends JarHandlerBase implements FileSystemInterface { FileUtil.copy(original, mirror); } catch (final IOException e) { - final String path1 = original.getPath(); - final String path2 = mirror.getPath(); + LOG.warn(e); + final String path = original.getPath(); + final String message = VfsBundle.message("jar.copy.error.message", path, mirror.getPath(), e.getMessage()); ApplicationManager.getApplication().invokeLater(new Runnable() { @Override public void run() { - Messages.showErrorDialog(VfsBundle.message("jar.copy.error.message", path1, path2, e.getMessage()), - VfsBundle.message("jar.copy.error.title")); + Messages.showErrorDialog(message, VfsBundle.message("jar.copy.error.title")); } }, ModalityState.NON_MODAL); - myFileSystem.setNoCopyJarForPath(path1); + myFileSystem.setNoCopyJarForPath(path); return original; } diff --git a/platform/platform-resources-en/src/messages/VfsBundle.properties b/platform/platform-resources-en/src/messages/VfsBundle.properties index 025d730802e2..7e7bf754e963 100644 --- a/platform/platform-resources-en/src/messages/VfsBundle.properties +++ b/platform/platform-resources-en/src/messages/VfsBundle.properties @@ -5,7 +5,7 @@ file.read.error=Cannot read from file {0}. file.write.error=Cannot write to file {0}. file.delete.root.error=Cannot delete root file {0}. jar.copy.progress=Copying {0}... -jar.copy.error.message=Cannot copy {0} to {1}.\n{2}. +jar.copy.error.message=Cannot copy {0} to {1}.\nReason: {2}. jar.copy.error.title=Error Copying File jar.modification.not.supported.error=Cannot modify jar or zip file {0} file.synchronize.progress=Synchronizing files... From d15633f1970d43619b4110c0b5aeef0ab489f180 Mon Sep 17 00:00:00 2001 From: Dmitry Avdeev Date: Fri, 14 Sep 2012 17:50:18 +0400 Subject: [PATCH 3/5] Static descriptions are back --- .../com/intellij/codeInspection/InspectionProfileEntry.java | 4 ---- .../com/intellij/codeInspection/ex/InspectionToolWrapper.java | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/platform/lang-api/src/com/intellij/codeInspection/InspectionProfileEntry.java b/platform/lang-api/src/com/intellij/codeInspection/InspectionProfileEntry.java index 11baeb9e7221..2d235798a7ab 100644 --- a/platform/lang-api/src/com/intellij/codeInspection/InspectionProfileEntry.java +++ b/platform/lang-api/src/com/intellij/codeInspection/InspectionProfileEntry.java @@ -247,16 +247,12 @@ public abstract class InspectionProfileEntry { * Override this method to return a html inspection description. Otherwise it will be loaded from resources using ID. * * @return hard-code inspection description. - * @deprecated this won't work for inspections configured via {@link InspectionEP} */ @Nullable public String getStaticDescription() { return null; } - /** - * @deprecated this won't work for inspections configured via {@link InspectionEP} - */ @Nullable public String getDescriptionFileName() { return null; diff --git a/platform/lang-impl/src/com/intellij/codeInspection/ex/InspectionToolWrapper.java b/platform/lang-impl/src/com/intellij/codeInspection/ex/InspectionToolWrapper.java index 38f413544a85..8b5bcbe94167 100644 --- a/platform/lang-impl/src/com/intellij/codeInspection/ex/InspectionToolWrapper.java +++ b/platform/lang-impl/src/com/intellij/codeInspection/ex/InspectionToolWrapper.java @@ -159,7 +159,7 @@ public abstract class InspectionToolWrapper Date: Fri, 14 Sep 2012 16:01:19 +0200 Subject: [PATCH 4/5] really test --- .../CloneDeclaresCloneNotSupportedInspectionTest.java | 10 ++++++++++ 1 file changed, 10 insertions(+) create mode 100644 plugins/InspectionGadgets/testsrc/com/siyeh/ig/cloneable/CloneDeclaresCloneNotSupportedInspectionTest.java diff --git a/plugins/InspectionGadgets/testsrc/com/siyeh/ig/cloneable/CloneDeclaresCloneNotSupportedInspectionTest.java b/plugins/InspectionGadgets/testsrc/com/siyeh/ig/cloneable/CloneDeclaresCloneNotSupportedInspectionTest.java new file mode 100644 index 000000000000..5b88db3d2659 --- /dev/null +++ b/plugins/InspectionGadgets/testsrc/com/siyeh/ig/cloneable/CloneDeclaresCloneNotSupportedInspectionTest.java @@ -0,0 +1,10 @@ +package com.siyeh.ig.cloneable; + +import com.siyeh.ig.IGInspectionTestCase; + +public class CloneDeclaresCloneNotSupportedInspectionTest extends IGInspectionTestCase { + + public void test() throws Exception { + doTest("com/siyeh/igtest/cloneable/clone_declares_clone_not_supported", new CloneDeclaresCloneNotSupportedInspection()); + } +} \ No newline at end of file From 1cd447d722cd29ea525f158f3ae63877a3dff4a3 Mon Sep 17 00:00:00 2001 From: "Denis.Zhdanov" Date: Fri, 14 Sep 2012 18:02:39 +0400 Subject: [PATCH 5/5] IDEA-91493 Conflict shortcut in Emacs keymap of "comment" and "undo" --- platform/platform-resources/src/idea/Keymap_Emacs.xml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/platform/platform-resources/src/idea/Keymap_Emacs.xml b/platform/platform-resources/src/idea/Keymap_Emacs.xml index 5f0ff589a373..730826c46f5a 100644 --- a/platform/platform-resources/src/idea/Keymap_Emacs.xml +++ b/platform/platform-resources/src/idea/Keymap_Emacs.xml @@ -268,6 +268,9 @@ + + + @@ -279,6 +282,7 @@ +