On 2/24/11 5:17 PM, Kumar Srinivasan wrote:
All the changes look good, the regression test
jdk/test/tools/pack200 and jdk/test/tools/jar
must be run, as jprt does not run these by default,
also suggest a full control build using jdk, deploy and install.
Thanks for looking at the changes. What I'll do is separate out the pack200
changes and file a bug and create a separate webrev for them. That way I can
push the other changes while we do the additional testing for pack200.
So, if intrinsic.properties isn't found, what should happen? Ignore this
(seems like a bad idea); issue warning message (how?); throw something like
FileNotFoundException? Well, static initializers can't throw checked
exceptions, so maybe throw a RuntimeException with a suitable message?
RE will be fine here, as you have already detected this condition should never
happen
unless there was a build issue.
OK, I'll have it throw a RuntimeException with a suitable message.
Thanks. I'll let you know how the testing goes.
s'marks
Thanks
Kumar
Advice appreciated.
s'marks
