RFR of 8129632: (fs) Files.probeContentType returns null on Mac OS X

[Brian Burkhalter]

On Jun 30, 2015, at 8:33 AM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:

>> Minor nits:
>> 
>> - getExtension has been moved to AbstractFileTypeDetector, can it be changed to be non-static? Also "Gets the extension" would be nicer as "Returns the extension”.
> 
> Will make those changes.
> 
>> - In the test then you are using equals("Mac OS X") the Apple recommendation is to use contains("OS X"). This may up during the Mac port a few years ago.
> 
> I’ll change it.

Here is an updated patch with the foregoing changes applied:

http://cr.openjdk.java.net/~bpb/8129632/webrev.02/

Thanks,

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