commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Frank W. Zammetti" <>
Subject Re: [io] FileSystemUtils
Date Tue, 22 Nov 2005 21:41:57 GMT
Would there be, or should I ask, how strong, objections to just changing
it to return BigInteger, period?  No overloaded versions, no
artificially-meaningful (although perfectly reasonable) "special" values,
etc.?  I know it's a change to the public interface and that's not
generally something that should be done, but it's a fairly new release,
can we get away with it this time?

If no one beats me to it, I'll try and open a ticket and submit a patch in
the next day or two, pending thoughts on the question above.

Frank W. Zammetti
Founder and Chief Software Architect
Omnytex Technologies
AIM: fzammetti
Yahoo: fzammetti

On Tue, November 22, 2005 4:38 pm, Stephen Colebourne said:
> This should be raised as a bug in bugzilla, as the existing code will
> just break if the free space is too large. I propose it should return -1
> if the free space is too large.
> To deal with large drives, we should add either a BigInteger returning
> method, or a freeSpaceMB method (that returns the value in MB!)
> Patches welcomed to the bugzilla call...
> Stephen
> Tim Roberts wrote:
>> You may be able to use BigInteger (arbitrary precision).
>> -----Original Message-----
>> From: Frank W. Zammetti []
>> Sent: 22 November 2005 20:35
>> To: Jakarta Commons Developers List
>> Cc:
>> Subject: Re: FileSystemUtils
>> Your welcome :) (I wrote that function :) )
>> I think your right, a double version would certainly be appropriate.
>> Kind
>> of ashamed I missed that actually :(
>> I wonder though, what would be the down-side to just having a double
>> version only? (aside from backward-compatibility at this point)... Would
>> it really result (potentially) in a loss of precision?
>> Feel free to submit a patch, it seems, to me anyway, like an obvious
>> thing
>> to commit, although I am not a committer, so you'll have to convince
>> others :)
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message