RFR: 8316304: (fs) Add support for BasicFileAttributes.creationTime() for Linux [v5]

Severin Gehwolf sgehwolf at openjdk.org
Tue Oct 3 11:39:37 UTC 2023


On Tue, 3 Oct 2023 10:09:30 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Severin Gehwolf has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add support for nano seconds for btime on Linux
>
> src/java.base/unix/classes/sun/nio/fs/UnixFileAttributes.java line 28:
> 
>> 26: package sun.nio.fs;
>> 27: 
>> 28: import java.nio.file.attribute.BasicFileAttributes;
> 
> I assume you'll update the copyright date on this file.

OK.

> test/jdk/java/nio/file/attribute/BasicFileAttributeView/CreationTime.java line 140:
> 
>> 138:             return (boolean)supportsBirthTime.invoke(null);
>> 139:         } catch (Throwable e) {
>> 140:             throw new RuntimeException("Illegal state!");
> 
> I agree with Brian, we need to find a better way to do this.

Sure, happy to. Any suggestions?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15792#discussion_r1343956701
PR Review Comment: https://git.openjdk.org/jdk/pull/15792#discussion_r1343955999


More information about the nio-dev mailing list