Alan,

i guess you're right, i have updated code to use the same method with fallback for creation .pack temporary file. New webrev can be found at http://cr.openjdk.java.net/~kizune/8020802/webrev.05

/Alex

On 10/22/13 23:55, Alan Bateman wrote:
On 22/10/2013 17:08, Alexander Zuev wrote:
Kumar,

new version of webrev where i have fixed all the coding style glitches i've found is here:
http://cr.openjdk.java.net/~kizune/8020802/webrev.04
I looked at the latest webrev and it looks much cleaner than the original patch (thanks).

One thing that is a bit inconsistent is that it has a fallback when it can't create <tmpbase>.jar in the temporary file but there isn't any fallback when it can't create <tmpbase>.pack.

Otherwise I think it's okay.

-Alan.


Reply via email to