Re: RFR: 7017818 NLS: JConsoleResources.java cannot be handled by translation team

2012-05-22 Thread Mandy Chung
Erik, I approve what you have - please go through and fix the formatting nits. I inlined my comment below and you can follow up them later if needed. On 5/22/2012 12:40 PM, Erik Gahlin wrote: Thanks for reviewing, I got a lot of unused import warnings from the IDE when I changed the Resour

Re: RFR: 7017818 NLS: JConsoleResources.java cannot be handled by translation team

2012-05-22 Thread Michael Fang
Thanks Erik. I have already downloaded the raw message.properties file from the webrev on 5/16 to proceed with the 7u6 translation. I performed a diff with current webrev, and it consists of mostly deleted resources and 1 renamed key CONNECTION_LOST1 => CONNECTION_LOST. I am fine with the ch

Re: RFR: 7017818 NLS: JConsoleResources.java cannot be handled by translation team

2012-05-22 Thread Erik Gahlin
Mandy Chung skrev 2012-05-21 23:13: Erik, I like your idea of trying to define the string constants for the keys to benefit from the compiler checking and catch any issue due to typos at compile time. My review comments below. Your webrev includes other cleanup. While it's good to do the simp