Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
MIME-Version: 1.0
In-Reply-To: <CACZkXPy0LNVJMyc9ksS7+1LRdGO1y3RuL_uGVB_cUuD_44JLUA@mail.gmail.com>
References: <CACZkXPy0LNVJMyc9ksS7+1LRdGO1y3RuL_uGVB_cUuD_44JLUA@mail.gmail.com>
From: Gary Gregory <garydgregory@gmail.com>
Date: Wed, 15 Nov 2017 10:33:35 -0700
Message-ID: <CACZkXPx0iF2vQpA=NKqGFVDi7XBbMqn+kuwS89z3y8hOcQXSAQ@mail.gmail.com>
Subject: Re: [io] New enum for file system info?
To: Commons Developers List <dev@commons.apache.org>
Content-Type: multipart/alternative; boundary="001a113dc79643e300055e08e7ca"
archived-at: Wed, 15 Nov 2017 17:33:41 -0000

--001a113dc79643e300055e08e7ca
Content-Type: text/plain; charset="UTF-8"

On Wed, Nov 15, 2017 at 10:30 AM, Gary Gregory <garydgregory@gmail.com>
wrote:

> I find myself writing and using non-OO code around things like:
>
>     private final int MAX_FILE_NAME_LENGTH_WINDOWS = 255;
>     private final int MAX_FILE_NAME_LENGTH_LINUX = 255;
>     private final int MAX_FILE_NAME_LENGTH_MAC = 255;
>     private final int MAX_FILE_NAME_LENGTH_MAC_OS9 = 31;
>
>     private final int MAX_FILE_PATH_LENGTH_WINDOWS = 32000;
>     private final int MAX_FILE_PATH_LENGTH_LINUX = 4096;
>     private final int MAX_FILE_PATH_LENGTH_MAC = 1024;
>
> But that is not even right for older Macs which limits file names to 31
> chars.
>
> I was thinking of creating a new enum:
>
> package org.apache.commons.io;
>
> public enum FileSystem {
>
>     LINUX(255, 4096),
>     MAC_OSX(255, 1024),
>     MAC_OSX_9(31, 1024),
>     WINDOWS(255, 32000);
>
>     private final long maxFileLength;
>     private final long maxPathLength;
>
>     private FileSystem(long maxFileLength, long maxPathLength) {
>         this.maxFileLength = maxFileLength;
>         this.maxPathLength = maxPathLength;
>     }
>
>     public long getMaxFileLength() {
>         return maxFileLength;
>     }
>
>     public long getMaxPathLength() {
>         return maxPathLength;
>     }
>
>     public boolean isIllegalFileName(char ch) {
>         ...
>     }
>
>     public String toLegalFileName(String candidate, char replacement) {
>         ...
>     }
>
> }
>
> I would also move the new method from https://issues.apache.org/
> jira/browse/IO-555 there and rename it "isIllegalFileName(char)"
>
> Thoughts?
>

The enum would obviously need a "public static FileSystem getCurrent()"
method.

Gary


>
> Gary
>

--001a113dc79643e300055e08e7ca--