OpenJDK / jdk / jdk
changeset 17484:4ad440adc4b1
8014676: Java debugger may fail to run
Summary: The problem is observed when the binaries for windows are placed under a path which contains a space
Reviewed-by: sla, alanb
Contributed-by: ivan.gerasimov@oracle.com
| author | dmeetry |
|---|---|
| date | Thu, 16 May 2013 19:28:44 +0400 |
| parents | ed57f2f26fb6 |
| children | 8708a3d0279d |
| files | jdk/src/share/classes/com/sun/tools/jdi/AbstractLauncher.java jdk/src/share/classes/com/sun/tools/jdi/SunCommandLineLauncher.java |
| diffstat | 2 files changed, 2 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/jdk/src/share/classes/com/sun/tools/jdi/AbstractLauncher.java Thu May 16 04:30:45 2013 -0700 +++ b/jdk/src/share/classes/com/sun/tools/jdi/AbstractLauncher.java Thu May 16 19:28:44 2013 +0400 @@ -142,7 +142,7 @@ * This class simply provides a context for a single launch and * accept. It provides instance fields that can be used by * all threads involved. This stuff can't be in the Connector proper - * because the connector is is a singleton and not specific to any + * because the connector is a singleton and is not specific to any * one launch. */ private class Helper {
--- a/jdk/src/share/classes/com/sun/tools/jdi/SunCommandLineLauncher.java Thu May 16 04:30:45 2013 -0700 +++ b/jdk/src/share/classes/com/sun/tools/jdi/SunCommandLineLauncher.java Thu May 16 19:28:44 2013 +0400 @@ -213,7 +213,7 @@ exePath = exe; } // Quote only if necessary in case the quote arg value is bogus - if (hasWhitespace(exe)) { + if (hasWhitespace(exePath)) { exePath = quote + exePath + quote; }