CMake replacing Autotools?

2021-03-17 Thread Christoph GrĂ¼ninger
Dear Java devs,

what do you think about the idea to replace Autotools by CMake? Is there
any chance anybody would be interested?
Would it be worth to write a feature request? Or even a JEP?

If somebody is interested I could elaborate more about why I think CMake
has advantages over Autotools.

Bye
Christoph


Integrated: 8263677: Improve Character.isLowerCase/isUpperCase lookups

2021-03-17 Thread Claes Redestad
On Tue, 16 Mar 2021 12:51:02 GMT, Claes Redestad  wrote:

> This patch changes the otherLowercase / otherUppercase bits to be set if 
> either the codepoint is of type LOWERCASE_LETTER and UPPERCASE_LETTER, or the 
> Unicode Other_Lowercase / Other_Uppercase property is set. This simplifies 
> the lookup in Character.isLowerCase/isUpperCase to a single table lookup, 
> which appears to be healthy for performance.
> 
> I also took the opportunity to clean up the somewhat dated GenerateCharacter 
> utility class.
> 
> Testing: tier1-3

This pull request has now been integrated.

Changeset: e152cc03
Author:Claes Redestad 
URL:   https://git.openjdk.java.net/jdk/commit/e152cc03
Stats: 283 lines in 8 files changed: 15 ins; 137 del; 131 mod

8263677: Improve Character.isLowerCase/isUpperCase lookups

Reviewed-by: erikj, ihse, naoto, rriggs

-

PR: https://git.openjdk.java.net/jdk/pull/3028


Re: RFR: 8263677: Improve Character.isLowerCase/isUpperCase lookups [v2]

2021-03-17 Thread Roger Riggs
On Tue, 16 Mar 2021 21:02:28 GMT, Claes Redestad  wrote:

>> This patch changes the otherLowercase / otherUppercase bits to be set if 
>> either the codepoint is of type LOWERCASE_LETTER and UPPERCASE_LETTER, or 
>> the Unicode Other_Lowercase / Other_Uppercase property is set. This 
>> simplifies the lookup in Character.isLowerCase/isUpperCase to a single table 
>> lookup, which appears to be healthy for performance.
>> 
>> I also took the opportunity to clean up the somewhat dated GenerateCharacter 
>> utility class.
>> 
>> Testing: tier1-3
>
> Claes Redestad has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Roger review + additional cleanups

Marked as reviewed by rriggs (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/3028


Integrated: 8263667: Avoid running GitHub actions on branches named pr/*

2021-03-17 Thread Robin Westberg
On Tue, 16 Mar 2021 10:53:06 GMT, Robin Westberg  wrote:

> When the Skara feature "dependent pull requests" is activated for the JDK 
> repository, branches with the name "pr/" will start to appear. These 
> will not be synced into personal forks by the Skara sync command, but if they 
> are synced manually, we should avoid running GitHub actions workflows on them.

This pull request has now been integrated.

Changeset: 86e9cd98
Author:Robin Westberg 
URL:   https://git.openjdk.java.net/jdk/commit/86e9cd98
Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod

8263667: Avoid running GitHub actions on branches named pr/*

Reviewed-by: ehelin, erikj, ihse

-

PR: https://git.openjdk.java.net/jdk/pull/3024