8206448: (fs) Extended attributes assumed to be enabled on ext3 (lnx)

Brian Burkhalter brian.burkhalter at oracle.com
Fri Jul 13 15:06:14 UTC 2018


On Jul 13, 2018, at 7:10 AM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:

>> This mostly looks okay to me. It would be good to move getKernelVersion closer to the end of the source file as this area puts the fields at the top.
> 
> I was going by the old coding style of putting static things at the top of the class but I agree it would be better near where it is used.

http://cr.openjdk.java.net/~bpb/8206448/webrev.01/

Updated to move getKernelVersion() to just above supportsFileAttributeView(). No update to xattr{Checked,Enabled}.

Thanks,

Brian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20180713/b9aa42c0/attachment.html>


More information about the nio-dev mailing list