Hi!

In major, this patch:
- removes some unused fields and hides other fields;
- does not treat '\\' (which starts an escape sequence) as a file separator on 
Unix;
- makes beginWithRootPathPrefix("c:rel_path") return 2 on Windows;
- makes beginWithRootPathPrefix("\\\\server\\share\\path") return >5 on Windows;
- updates the documentation.

ChangeLog entries:
        * gnu/java/io/PlatformHelper.java:
        PlatformHelper: Make class final.
        (isWindows): Remove field (the field is not used in Classpath).
        (pathSeparator): Likewise.
        (pathSeparatorChar): Likewise.
        (separator): Make private instead of public (the field is not used in
        Classpath outside this class); use SystemProperties.
        (separatorChar): Likewise.
        (PlatformHelper()): New private constructor (to prohibit
        instantiation).
        (beginWithRootPathPrefix(String)): Use platform-specific separator
        instead of a backslash character (since the latter denotes an escape
        sequence on Unix).
        (endWithSeparator(String)): Likewise.
        (lastIndexOfSeparator(String)): Likewise.
        (beginWithRootPathPrefix(String)): Recognize "d:" and
        "\\server\share\" prefixes on a Win32 platform; use "separatorChar"
        to check for a non-Unix platform; recognize NetWare-specific path
        formats.
        (beginWithRootPathPrefix(String)): Update documentation.
        (isRootDirectory(String)): Likewise.
        (lastIndexOfSeparator(String)): Optimize the implementation.

Regards.

Attachment: classpath-ivmai-34.diff
Description: Binary data

Reply via email to