Re: RFR: 8271302: Regex Test Refresh [v4]

2021-08-27 Thread Stuart Marks
On Fri, 20 Aug 2021 21:17:50 GMT, Ian Graves wrote: >> 8271302: Regex Test Refresh > > Ian Graves has updated the pull request incrementally with one additional > commit since the last revision: > > Additional cleanup Marked as reviewed by smarks (Reviewer). Whew! Changes to GraphemeTest.ja

Re: RFR: 8271603: Unnecessary Vector usage in java.desktop [v4]

2021-08-27 Thread Sergey Bylokhov
On Fri, 27 Aug 2021 17:52:19 GMT, Andrey Turbanov wrote: >> src/java.desktop/share/classes/javax/swing/JMenu.java line 1340: >> >>> 1338: } >>> 1339: MenuElement[] me = new MenuElement[elements.size()]; >>> 1340: elements.toArray(me); >> >> Implementation of the "toArra

Re: RFR: 8271603: Unnecessary Vector usage in java.desktop [v4]

2021-08-27 Thread Andrey Turbanov
On Fri, 27 Aug 2021 17:04:33 GMT, Sergey Bylokhov wrote: >> Andrey Turbanov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8271603: Unnecessary Vector usage in java.desktop >> migrate even more usages > > src/java.desktop/share/classe

Re: RFR: 8271603: Unnecessary Vector usage in java.desktop [v4]

2021-08-27 Thread Sergey Bylokhov
On Wed, 25 Aug 2021 17:53:00 GMT, Andrey Turbanov wrote: >> Usage of thread-safe collection `Vector` is unnecessary. It's recommended to >> use `ArrayList` if a thread-safe implementation is not needed. In >> post-BiasedLocking times, this is gets worse, as every access is >> synchronized. >>