Updated webrev to include Mandy’s and Alan’s comments: http://cr.openjdk.java.net/~chegar/8039362/02/webrev/
On 7 Apr 2014, at 22:50, Mandy Chung <mandy.ch...@oracle.com> wrote: > …. > > line 225: since content-types.properties is moved to the same package as > MimeType class, you can simply use the relative path > "content-types.properties" to get the resource stream Done. > line 240: should we expect it always exist? Perhaps an assert. I decided to throw InternalError. There is a code or build problem if this ever happens > I wonder if this can be refactored so that it can use try-with-resources. Done. -Chris. > thanks > Mandy
