[Bug 57310] New: Replace System.getProperty("file.separator") with File.separator throughout

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=57310

            Bug ID: 57310
           Summary: Replace System.getProperty("file.separator") with
                    File.separator throughout
           Product: JMeter
           Version: 2.12
          Hardware: PC
                OS: Mac OS X 10.4
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: sebb@apache.org

Most of JMeter uses File.separator, but there are a few instances of
System.getProperty("file.separator")

The two can be different, but still allow the JVM to start up.
I found the following worked:

java -Dfile.separator=/xyz

for Unix or the equivalent \xyz on Windows.

Using a different first character does not work - the JVM usually throws an
exception as it cannot find some required files if the sep. is incorrect.

It looks like only the first character of the property is used by the JVM.
However, this is not done by the application code I have seen, which means that
there could be a discrepancy between the two methods.

File.separator has the additional benefit that it is not subject to a privilege
check.

The same applies to "path.separator" => File.pathSeparator

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 57310] Replace System.getProperty("file.separator") with File.separator throughout

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=57310

Sebb <se...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Sebb <se...@apache.org> ---
URL: http://svn.apache.org/r1643364
Log:
Replace System.getProperty("file.separator") with File.separator throughout
(Also "path.separator" with File.pathSeparator)
Bugzilla Id: 57310

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/NewDriver.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/util/FileDialoger.java
    jmeter/trunk/src/jorphan/org/apache/jorphan/reflect/ClassFinder.java
    jmeter/trunk/xdocs/changes.xml

-- 
You are receiving this mail because:
You are the assignee for the bug.