Re: RFR: JDK-8221749: Error messages

2019-04-09 Thread Alexander Matveev
Hi Andy, webrev.02 looks good. Thanks, Alexander On 4/9/2019 4:45 AM, Andy Herrick wrote: On 4/8/2019 6:05 PM, Andy Herrick wrote: On 4/8/2019 5:41 PM, Alexander Matveev wrote: Hi Andy, Looks good. Also, many messages (including added in this review) in MainResources.properties do have

Re: RFR: JDK-8221749: Error messages

2019-04-09 Thread Andy Herrick
On 4/8/2019 6:05 PM, Andy Herrick wrote: On 4/8/2019 5:41 PM, Alexander Matveev wrote: Hi Andy, Looks good. Also, many messages (including added in this review) in MainResources.properties do have "." at the end and some don't. I think we should cleanup it and add "." if needed. Yes - som

Re: RFR: JDK-8221749: Error messages

2019-04-08 Thread Andy Herrick
On 4/8/2019 5:41 PM, Alexander Matveev wrote: Hi Andy, Looks good. Also, many messages (including added in this review) in MainResources.properties do have "." at the end and some don't. I think we should cleanup it and add "." if needed. Yes - some messages are not full sentences, but most

Re: RFR: JDK-8221749: Error messages

2019-04-08 Thread Alexander Matveev
Hi Andy, Looks good. Also, many messages (including added in this review) in MainResources.properties do have "." at the end and some don't. I think we should cleanup it and add "." if needed. Thanks, Alexander On 4/8/2019 4:41 AM, Andy Herrick wrote: Please review the jpackage fix for bug

RFR: JDK-8221749: Error messages

2019-04-08 Thread Andy Herrick
Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). [1] http://cr.openjdk.java.net/~herrick/8221749/ [2] https://bugs.openjdk.java.net/browse/JDK-8221749/ /Andy