Re: RFR: 8268339: Upstream: 8267989: Exceptions thrown during upcalls should be handled (Pt. 2)

2021-06-08 Thread openjdk-notifier [bot]
On Mon, 7 Jun 2021 16:38:01 GMT, Jorn Vernee wrote: > Hi, > > This is part 2 of a two part upstreaming process of the patch mentioned in > the subject line. The patch was split into 2 in order to document 2 separate > specification changes that arose from a change in the implementation, with

Re: RFR: 8267521: Post JEP 411 refactoring: maximum covering > 50K [v3]

2021-06-02 Thread openjdk-notifier [bot]
On Fri, 21 May 2021 20:37:30 GMT, Weijun Wang wrote: >> The code change refactors classes that have a `SuppressWarnings("removal")` >> annotation that covers more than 50KB of code. The big annotation is often >> quite faraway from the actual deprecated API usage it is suppressing, and >>

Re: RFR: 8278065: Refactor subclassAudits to use ClassValue

2021-12-10 Thread openjdk-notifier[bot]
On Wed, 1 Dec 2021 14:45:23 GMT, Roman Kennke wrote: > As a follow-up to #6375, this change refactors > java.io.ObjectInputStream.Caches#subclassAudits and > java.io.ObjectOutputStream.Caches#subclassAudits to use ClassValue instead of > SoftReference, similar to what we did in #6375 for >

Re: RFR: 8284856: Add a test case for checking UnicodeScript entity numbers [v2]

2022-04-18 Thread openjdk-notifier[bot]
On Thu, 14 Apr 2022 22:27:20 GMT, Naoto Sato wrote: >> Added the test case, and eliminated the immediate hashmap value, replaced >> with the ordinal number of `Character.UnicodeScript.UNKNOWN`. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the

Re: RFR: 8262889: Compiler implementation for Record Patterns [v5]

2022-05-16 Thread openjdk-notifier[bot]
On Tue, 10 May 2022 09:57:48 GMT, Jan Lahoda wrote: >> 8262889: Compiler implementation for Record Patterns >> >> A first version of a patch that introduces record patterns into javac as a >> preview feature. For the specification, please see: >>

Re: RFR: 8283689: Update the foreign linker VM implementation [v16]

2022-05-12 Thread openjdk-notifier[bot]
On Thu, 12 May 2022 16:01:17 GMT, Jorn Vernee wrote: >> Hi, >> >> This PR updates the VM implementation of the foreign linker, by bringing >> over commits from the panama-foreign repo. >> >> This is split off from the main JEP integration for 19, since we have >> limited resources to handle

Re: RFR: 8286669: Replace MethodHandle specialization with ASM in mainline [v4]

2022-05-18 Thread openjdk-notifier[bot]
On Tue, 17 May 2022 16:22:15 GMT, Jorn Vernee wrote: >> Hi, >> >> This PR brings over commits from the panama-foreign repo. These commits >> mostly pertain to the switch to ASM and away from MethodHandle combinators >> for the binding recipe specialization. But, there is one more commit which

Re: RFR: 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved flags is used [v3]

2022-06-07 Thread openjdk-notifier[bot]
On Tue, 7 Jun 2022 13:30:50 GMT, Thiago Henrique Hüpner wrote: >> 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved >> flags is used > > Thiago Henrique Hüpner has refreshed the contents of this pull request, and > previous commits have been removed. The incremental

Re: RFR: 8285838: DST not applying properly with zone id offset set with TZ env variable [v6]

2022-06-07 Thread openjdk-notifier[bot]
On Tue, 7 Jun 2022 14:11:57 GMT, Gaurav Chaudhari wrote: >> Gaurav Chaudhari has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - 8285838: Cleanup of trailing whitespace >> - 8285838: Corrected month comparison check for TZ DST > >

Re: RFR: 8284493: Fix rounding error in computeNextExponential [v2]

2022-06-06 Thread openjdk-notifier[bot]
On Thu, 2 Jun 2022 03:00:32 GMT, Chris Hennick wrote: >> Chris Hennick has refreshed the contents of this pull request, and previous >> commits have been removed. The incremental views will show differences >> compared to the previous content of the PR. The pull request contains one >> new

Re: RFR: 8287353: Use snippet tag instead of pre tag in Javadoc of InterruptedException

2022-06-06 Thread openjdk-notifier[bot]
On Wed, 27 Apr 2022 13:33:29 GMT, Thiago Henrique Hüpner wrote: >> 8287353: Use snippet tag instead of pre tag in Javadoc of >> InterruptedException > > If this is a valid change, could someone please create a Jira issue to link > to it? @Thihup Please do not rebase or force-push to an