RFR: 8335108: Build error after JDK-8333658 due to class templates

David Holmes dholmes at openjdk.org
Wed Jul 3 05:21:21 UTC 2024


On Tue, 25 Jun 2024 18:52:20 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:

> Hi all, 
> 
> This PR addresses [8335108](https://bugs.openjdk.org/browse/JDK-8335108). 
> 
> The error arises as template-id is not allowed for constructor/destructor in C++20. 
> 
> Testing: 
> - [x] Compilation succeeds with g++ 14.1.1.
> 
> Thanks, 
> Sonia

The 24-hour rule is mentioned here:

https://openjdk.org/guide/#life-of-a-pr

6. Allow enough time for review

In general all PRs should be open for at least 24 hours to allow for reviewers in all time zones to get a chance to see it. It may actually happen that even 24 hours isn’t enough. Take into account weekends, holidays, and vacation times throughout the world and you’ll realize that a change that requires more than just a trivial review may have to be open for a while. In some areas [trivial] changes are allowed to be pushed without the 24 hour delay. Ask your reviewers if you think this applies to your change.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/19890#issuecomment-2205119100


More information about the hotspot-dev mailing list