Re: RFR: JDK-8303798: REDO - Remove fdlibm C sources

2023-04-02 Thread Alan Bateman
On Sat, 1 Apr 2023 18:08:44 GMT, Joe Darcy wrote: > This PR is a redo of JDK-8302801: Remove fdlibm C sources. The problem with > JDK-8302801 was that it neglected (mea culpa) to include a Java > implementation of IEEEremainder before the FDLIBM C implementation was > deleted. Such an implemen

Re: RFR: JDK-8304945: StringBuilder and StringBuffer should implement Appendable explicitly

2023-04-02 Thread Tingjun Yuan
On Sat, 1 Apr 2023 17:34:37 GMT, Joe Darcy wrote: > The StringBuilder and StringBuffer classes are Appendable by virtue of from > subclasses their non-public superclass AbstractStringBuilder. > > It is slightly clearer to declare StringBuilder and StringBuffer to directly > implement Appendabl

Re: RFR: 8305400: ISO 4217 Amendment 175 Update

2023-04-02 Thread M4ximumPizza
On Fri, 31 Mar 2023 21:38:31 GMT, Justin Lu wrote: > Please review the ISO 4217 amendment 175 update. > > There are no meaningful code changes, but the version number should be > updated accordingly to be in sync. Marked as reviewed by m4ximumpi...@github.com (no known OpenJDK username).

Re: RFR: 8304450: [vectorapi] Refactor VectorShuffle implementation [v6]

2023-04-02 Thread Xiaohong Gong
On Fri, 31 Mar 2023 12:25:16 GMT, Quan Anh Mai wrote: >> Hi, >> >> This patch reimplements `VectorShuffle` implementations to be a vector of >> the bit type. Currently, VectorShuffle is stored as a byte array, and would >> be expanded upon usage. This poses several drawbacks: >> >> 1. Ineffic

Re: RFR: 8304014: Convert test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java to junit [v6]

2023-04-02 Thread Lance Andersen
On Fri, 31 Mar 2023 19:59:10 GMT, Eirik Bjorsnos wrote: >> CorruptedZipFiles could benefit from some spring cleaning and a conversion >> to junit: >> >> - The actual tests are moved into their own `@Test` methods, given more >> meaningful names and a Javadoc comment explaining the constraint b

Re: RFR: JDK-8304945: StringBuilder and StringBuffer should implement Appendable explicitly

2023-04-02 Thread M4ximumPizza
On Sat, 1 Apr 2023 17:34:37 GMT, Joe Darcy wrote: > The StringBuilder and StringBuffer classes are Appendable by virtue of from > subclasses their non-public superclass AbstractStringBuilder. > > It is slightly clearer to declare StringBuilder and StringBuffer to directly > implement Appendabl

Re: RFR: JDK-8303798: REDO - Remove fdlibm C sources

2023-04-02 Thread Julian Waters
On Sat, 1 Apr 2023 18:08:44 GMT, Joe Darcy wrote: > This PR is a redo of JDK-8302801: Remove fdlibm C sources. The problem with > JDK-8302801 was that it neglected (mea culpa) to include a Java > implementation of IEEEremainder before the FDLIBM C implementation was > deleted. Such an implemen

Re: RFR: 8304014: Convert test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java to junit [v5]

2023-04-02 Thread Lance Andersen
On Fri, 31 Mar 2023 19:58:24 GMT, Eirik Bjorsnos wrote: >> test/jdk/java/util/zip/ZipFile/CorruptedZipFiles.java line 192: >> >>> 190: /* >>> 191: * Validate that a ZipException is thrown when the 'End of Central >>> Directory' >>> 192: * (END) header has a CEN offset incoherent w

Re: RFR: JDK-8303798: REDO - Remove fdlibm C sources

2023-04-02 Thread Iris Clark
On Sat, 1 Apr 2023 18:08:44 GMT, Joe Darcy wrote: > This PR is a redo of JDK-8302801: Remove fdlibm C sources. The problem with > JDK-8302801 was that it neglected (mea culpa) to include a Java > implementation of IEEEremainder before the FDLIBM C implementation was > deleted. Such an implemen

Re: RFR: 8303762: [vectorapi] Intrinsification of Vector.slice [v4]

2023-04-02 Thread Quan Anh Mai
> `Vector::slice` is a method at the top-level class of the Vector API that > concatenates the 2 inputs into an intermediate composite and extracts a > window equal to the size of the inputs into the result. It is used in vector > conversion methods where the part number is not 0 to slice the pa

RFR: JDK-8303798: REDO - Remove fdlibm C sources

2023-04-02 Thread Joe Darcy
This PR is a redo of JDK-8302801: Remove fdlibm C sources. The problem with JDK-8302801 was that it neglected (mea culpa) to include a Java implementation of IEEEremainder before the FDLIBM C implementation was deleted. Such an implementation has been successfully provided under JDK-8304028: Por

Re: The non-deterministic iteration order of Immutable Collections

2023-04-02 Thread Chris Hegarty
Hi Stuart, First, thanks for you detailed reply. On 25/03/2023 00:16, Stuart Marks wrote: ... Yes, this has come up before, but it's been mostly theoretical. That is, people worry about this when they hear of the idea of randomized iteration order, but I've never heard any followup. This is

Re: RFR: JDK-8303798: REDO - Remove fdlibm C sources

2023-04-02 Thread Joe Darcy
On Sun, 2 Apr 2023 07:24:24 GMT, Alan Bateman wrote: > I assume at least tier1-4 has been run, in which case this looks good (same > as previous PR). Right; tier 1 - 4 job was successful other than an unrelated time-out. Previously, with the initial removal attempt there were many failures in

Re: RFR: JDK-8304945: StringBuilder and StringBuffer should implement Appendable explicitly

2023-04-02 Thread Joe Darcy
On Sun, 2 Apr 2023 06:45:30 GMT, Alan Bateman wrote: > Right now, the javadoc for SB lists Appendable in the "All Implemented > Interfaces" list. With this change it will be shown in the class declaration > in the list of "implements" list. Make sense. Correct; after the change, the appearance