On Fri, 23 Sep 2022 12:47:24 GMT, Magnus Ihse Bursie wrote:
>> Thanks for your review.
>>
>> Your mentioned solution should work and we can also enable the pragma only
>> for aarch64 + gcc>=12 condition.
>> I considered such a solution when preparing this patch.
>>
>> But I personally prefer t
> This warning seems to be a false positive, because 1) array "fq" with
> elements from index 0 to "jz" has already been initialized as "fw" at line
> 290 [1], and 2) variable "jz" should be non-negative from the comment at line
> 99 [2].
>
> Note-1: GCC warning option -Wmaybe-uninitialized is
On Thu, 25 Aug 2022 15:35:53 GMT, Michael Ernst wrote:
> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
I'm not a committer (yet) so I can't [sponsor](https://openjdk.org/sponsor/)
this PR, even though I would.
-
PR: https://git.openjdk.org/jdk/pull/10
On Tue, 30 Aug 2022 09:55:18 GMT, Lance Andersen wrote:
>> src/java.base/share/native/libzip/zlib/zlib.h line 756:
>>
>>> 754:If this is done, the old level and strategy will be applied to the
>>> data
>>> 755:compressed before deflateParams(), and the new level and strategy
>>> will b
On Tue, 30 Aug 2022 09:31:07 GMT, Alan Bateman wrote:
>> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
>
> src/java.base/share/native/libzip/zlib/zlib.h line 756:
>
>> 754:If this is done, the old level and strategy will be applied to the
>> data
>> 755:com
On Thu, 25 Aug 2022 15:35:53 GMT, Michael Ernst wrote:
> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
src/java.base/share/native/libzip/zlib/zlib.h line 756:
> 754:If this is done, the old level and strategy will be applied to the
> data
> 755:compressed
On Sun, 25 Sep 2022 16:51:27 GMT, SWinxy wrote:
> Oh no what happened
Oops! Thanks for pointing out the goof. It seems to be fixed now.
It has now been a month since I opened the pull request, and I have responded
to each substantive comment promptly. It would be great to merge this pull
r
On Thu, 25 Aug 2022 15:35:53 GMT, Michael Ernst wrote:
> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
Oh no what happened
![image](https://user-images.githubusercontent.com/8303399/192155378-4bb439fa-d27a-417d-8a3c-cf7f7329458f.png)
src/java.xml/share/classes/com/
On Sat, 24 Sep 2022 10:41:05 GMT, Jaikiran Pai wrote:
> Splitting these changes into smaller PRs instead of one big one that touches
> several different areas of the JDK and 108 files, will help reviewers.
OK. I'll repeat my comment from before:
> Feel free to split up the pull request, if yo
On Thu, 25 Aug 2022 15:35:53 GMT, Michael Ernst wrote:
> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
Hello Michael,
> > these changes are mostly fine.
>
> Can you be specific about the exact problems that you noticed that prevented
> you from saying "these chan
On Mon, 5 Sep 2022 14:54:50 GMT, Michael Ernst wrote:
> Can you be specific about the exact problems that you noticed that prevented
> you from saying "these changes are fine"?
These changes are fine. I don't see an instance where the duplicated words
would mean a change in the specification.
On Mon, 5 Sep 2022 06:32:44 GMT, Jaikiran Pai wrote:
> these changes are mostly fine.
Can you be specific about the exact problems that you noticed that prevented
you from saying "these changes are fine"? It doesn't seem useful to make a
pull request with known deficiencies.
> we usually spl
On Thu, 25 Aug 2022 19:55:47 GMT, Michael Ernst wrote:
>> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
>
>> If you already are an OpenJDK
>> [Author](https://openjdk.java.net/bylaws#author),
>> [Committer](https://openjdk.java.net/bylaws#committer) or
>> [Reviewe
On Thu, 25 Aug 2022 15:35:53 GMT, Michael Ernst wrote:
> 8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
> If you already are an OpenJDK
> [Author](https://openjdk.java.net/bylaws#author),
> [Committer](https://openjdk.java.net/bylaws#committer) or
> [Reviewer](htt
8294321: Fix typos in files under test/jdk/java, test/jdk/jdk, test/jdk/jni
-
Commit messages:
- Remove file that was removed upstream
- Fix inconsistency in capitalization
- Undo change in zlip
- Fix typos
Changes: https://git.openjdk.org/jdk/pull/10029/files
Webrev: https://we
On Sun, 25 Sep 2022 07:28:56 GMT, Alan Bateman wrote:
>>> I do not quite understand what would be wrong with the code below instead
>>> of falling back to the super implementation _in case of non-empty buffer_?
>>
>> I think you are asking if is safe to leak a reference to the internal
>> buff
On Tue, 2 Aug 2022 13:57:03 GMT, iaroslavski wrote:
>> Sorting:
>>
>> - adopt radix sort for sequential and parallel sorts on
>> int/long/float/double arrays (almost random and length > 6K)
>> - fix tryMergeRuns() to better handle case when the last run is a single
>> element
>> - minor javado
On Sun, 25 Sep 2022 07:28:56 GMT, Alan Bateman wrote:
> > @AlanBateman Kindly requesting approval. :-)
>
> I don't have any more comments, the recent test cleanups are good. I assume
> @bplb will sponsor.
Thanks a lot for all your kind help with this PR! :-)
-
PR: https://git.ope
On Sun, 11 Sep 2022 07:44:44 GMT, Alan Bateman wrote:
>> Markus KARG has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - corrected copyright
>> - testing transferTo() after reset()
>
>> I do not quite understand what would be wrong with t
19 matches
Mail list logo