Need review for 6873621: (file) FileStore.supportsFileAttributeView(Class<FileAttributeView> type) returns wrong result

Andrew John Hughes gnu_andrew at
Fri Sep 4 06:42:57 PDT 2009

2009/9/4 Alan Bateman <Alan.Bateman at>:
> I need a reviewer for this one - it's very simple.
> FileStore.supportsFileAttributeView(Class<? extends FileAttributeView>)
> should return true if the file store supports the given file attribute view.
> The method needs to be overridden in the Solaris and Linux implementations
> so that it returns the right answer for the more specialized views. Also,
> the Windows implementation is missing one case. The webrev is here:
> Thanks,
> Alan.

Looks good to me.  I like the idea of making this safer by checking
against a type rather than an arbitrary String.  This also makes these
classes match UnixFileStore which already does this for the

As an aside, I noticed that Solaris is the only implementation to use
AclFileAttributeView.  Don't both Linux and Windows have some form of
ACL support?  Something to fix long term I guess.
Andrew :-)

Free Java Software Engineer
Red Hat, Inc. (

Support Free Java!
Contribute to GNU Classpath and the OpenJDK

PGP Key: 94EFD9D8 (
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8

More information about the nio-dev mailing list