ggregory    2004/08/30 14:19:52

  Modified:    lang/src/java/org/apache/commons/lang SystemUtils.java
  Log:
  Refactor "Windows" into private static final String OS_NAME_WINDOWS_PREFIX = 
"Windows";
  
  Revision  Changes    Path
  1.34      +13 -8     
jakarta-commons/lang/src/java/org/apache/commons/lang/SystemUtils.java
  
  Index: SystemUtils.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/lang/src/java/org/apache/commons/lang/SystemUtils.java,v
  retrieving revision 1.33
  retrieving revision 1.34
  diff -u -r1.33 -r1.34
  --- SystemUtils.java  25 Feb 2004 00:25:29 -0000      1.33
  +++ SystemUtils.java  30 Aug 2004 21:19:52 -0000      1.34
  @@ -37,6 +37,11 @@
    */
   public class SystemUtils {
   
  +    /**
  +     * The prefix String for all Windows OS.
  +     */
  +    private static final String OS_NAME_WINDOWS_PREFIX = "Windows";
  +    
       // System property constants
       //-----------------------------------------------------------------------
       // These MUST be declared first. Other constants depend on this.
  @@ -974,7 +979,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS = getOSMatches("Windows");
  +    public static final boolean IS_OS_WINDOWS = 
getOSMatches(OS_NAME_WINDOWS_PREFIX);
   
       /**
        * <p>Is <code>true</code> if this is Windows 2000.</p>
  @@ -984,7 +989,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_2000 = getOSMatches("Windows", "5.0");
  +    public static final boolean IS_OS_WINDOWS_2000 = 
getOSMatches(OS_NAME_WINDOWS_PREFIX, "5.0");
   
       /**
        * <p>Is <code>true</code> if this is Windows 95.</p>
  @@ -994,7 +999,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_95 = getOSMatches("Windows 9", "4.0");
  +    public static final boolean IS_OS_WINDOWS_95 = 
getOSMatches(OS_NAME_WINDOWS_PREFIX + " 9", "4.0");
       // JDK 1.2 running on Windows98 returns 'Windows 95', hence the above
   
       /**
  @@ -1005,7 +1010,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_98 = getOSMatches("Windows 9", "4.1");
  +    public static final boolean IS_OS_WINDOWS_98 = 
getOSMatches(OS_NAME_WINDOWS_PREFIX + " 9", "4.1");
       // JDK 1.2 running on Windows98 returns 'Windows 95', hence the above
   
       /**
  @@ -1016,7 +1021,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_ME = getOSMatches("Windows", "4.9");
  +    public static final boolean IS_OS_WINDOWS_ME = 
getOSMatches(OS_NAME_WINDOWS_PREFIX, "4.9");
       // JDK 1.2 running on WindowsME may return 'Windows 95', hence the above
   
       /**
  @@ -1027,7 +1032,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_NT = getOSMatches("Windows NT");
  +    public static final boolean IS_OS_WINDOWS_NT = 
getOSMatches(OS_NAME_WINDOWS_PREFIX + " NT");
       // Windows 2000 returns 'Windows 2000' but may suffer from same JDK1.2 problem
   
       /**
  @@ -1038,7 +1043,7 @@
        * 
        * @since 2.0
        */
  -    public static final boolean IS_OS_WINDOWS_XP = getOSMatches("Windows", "5.1");
  +    public static final boolean IS_OS_WINDOWS_XP = 
getOSMatches(OS_NAME_WINDOWS_PREFIX, "5.1");
       // Windows XP returns 'Windows 2000' just for fun...
   
       //-----------------------------------------------------------------------    
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to