Codereview Request: 8011647: Add java.time.Instant methods to java.nio.file.attribute.FileTime

Xueming Shen xueming.shen at
Fri Apr 12 18:16:27 UTC 2013

On 04/12/2013 11:10 AM, Alan Bateman wrote:
> On 12/04/2013 18:28, Xueming Shen wrote:
>> I'm not sure if the 3-digit unit fitting is "better" or not. But xml spec cited in the
>> toString() API specifies that the "trailing zeros are optional" in its "lexical
>> representatin" section and "trailing zeros are prohibited..." in "canonical
>> representation" section. So I think trimming trailing zeros in FileTime.toString()
>> is just fine.
> The deviation and reference to the XML scheme spec was intended to be deal with the issues of year 0000 and > 9999. I'm happy with dropping the trailing zeros, I just wasn't sure if you suggesting that it would have been better to use 3-digit unit that allow trailing zeros.
>> :
>> Updated.
>> <>
> Looks fine. The eqTime method is still printing to System.out in the test but it's not a big deal.

Missed this one. Updated. I think we are good to go.


More information about the core-libs-dev mailing list