Re: RFR: 8295000: java/util/Formatter/Basic test cleanup [v2]

2022-10-27 Thread Justin Lu
On Thu, 27 Oct 2022 17:13:23 GMT, Lance Andersen wrote: >> Justin Lu has updated the pull request with a new target base due to a merge >> or a rebase. The incremental webrev excludes the unrelated changes brought >> in by the merge/rebase. The pull request contains four additional commits >>

Re: RFR: 8295000: java/util/Formatter/Basic test cleanup [v2]

2022-10-27 Thread Lance Andersen
On Wed, 26 Oct 2022 21:00:43 GMT, Justin Lu wrote: >> Issue: java/util/Formatter/Basic regression test emits lots of warning >> messages (~60). >> >> Fix: Made adjustments to Basic-X.java.template as the BasicXXX.java files >> where the errors originate from are generated from the template.

Re: RFR: 8295000: java/util/Formatter/Basic test cleanup [v2]

2022-10-26 Thread Naoto Sato
On Wed, 26 Oct 2022 21:00:43 GMT, Justin Lu wrote: >> Issue: java/util/Formatter/Basic regression test emits lots of warning >> messages (~60). >> >> Fix: Made adjustments to Basic-X.java.template as the BasicXXX.java files >> where the errors originate from are generated from the template.

Re: RFR: 8295000: java/util/Formatter/Basic test cleanup [v2]

2022-10-26 Thread Brent Christian
On Wed, 26 Oct 2022 21:00:43 GMT, Justin Lu wrote: >> Issue: java/util/Formatter/Basic regression test emits lots of warning >> messages (~60). >> >> Fix: Made adjustments to Basic-X.java.template as the BasicXXX.java files >> where the errors originate from are generated from the template.

Re: RFR: 8295000: java/util/Formatter/Basic test cleanup [v2]

2022-10-26 Thread Justin Lu
> Issue: java/util/Formatter/Basic regression test emits lots of warning > messages (~60). > > Fix: Made adjustments to Basic-X.java.template as the BasicXXX.java files > where the errors originate from are generated from the template. > > Note: The reason why there is white space added (and