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

Alan Bateman Alan.Bateman at oracle.com
Fri Jul 13 15:09:17 UTC 2018



On 13/07/2018 16:06, Brian Burkhalter wrote:
>
> On Jul 13, 2018, at 7:10 AM, Brian Burkhalter 
> <brian.burkhalter at oracle.com <mailto: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/ 
> <http://cr.openjdk.java.net/%7Ebpb/8206448/webrev.01/>
>
> Updated to move getKernelVersion() to just above 
> supportsFileAttributeView(). No update to xattr{Checked,Enabled}.
>
Looks okay to me.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20180713/66e200ea/attachment-0001.html>


More information about the nio-dev mailing list