Re: RFR: 8260265: UTF-8 by Default [v6]

2021-07-22 Thread Alan Bateman
On Wed, 21 Jul 2021 17:34:15 GMT, Naoto Sato  wrote:

>> This is an implementation for the `JEP 400: UTF-8 by Default`. The gist of 
>> the changes is `Charset.defaultCharset()` returning `UTF-8` and 
>> `file.encoding` system property being added in the spec, but another notable 
>> modification is in `java.io.PrintStream` where it continues to use the 
>> `Console` encoding as the default charset instead of `UTF-8`. Other changes 
>> are mostly clarification of the term "default charset" and their links. 
>> Corresponding CSR has also been drafted.
>> 
>> JEP 400: https://bugs.openjdk.java.net/browse/JDK-8187041
>> CSR: https://bugs.openjdk.java.net/browse/JDK-8260266
>
> Naoto Sato has updated the pull request with a new target base due to a merge 
> or a rebase. The incremental webrev excludes the unrelated changes brought in 
> by the merge/rebase. The pull request contains ten additional commits since 
> the last revision:
> 
>  - Merge branch 'master' into JDK-8260265
>  - Refined `file.encoding` description
>  - Merge branch 'master' into JDK-8260265
>  - Reflects comments
>  - Removed leftover `console` references in `PrintStream`
>  - Reflects comments
>  - Reflects review comments
>  - Merge branch 'master' into JDK-8260265
>  - 8260265: UTF-8 by Default

Thanks for incorporating the suggestion for the getProperties text. I think it 
looks good now.

-

Marked as reviewed by alanb (Reviewer).

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


Integrated: 8270286: com.sun.net.httpserver.spi.HttpServerProvider: remove use of deprecated API

2021-07-22 Thread Julia Boes
On Thu, 15 Jul 2021 09:24:17 GMT, Julia Boes  wrote:

> This is a cleanup fix that replaces the deprecated method 
> `Class::newInstance()` with `cls.getDeclaredConstructor().newInstance()` when 
> loading the provider.

This pull request has now been integrated.

Changeset: 50bb7313
Author:Julia Boes 
URL:   
https://git.openjdk.java.net/jdk/commit/50bb7313a75e6e573549637763881ae3368b3ca0
Stats: 172 lines in 2 files changed: 166 ins; 0 del; 6 mod

8270286: com.sun.net.httpserver.spi.HttpServerProvider: remove use of 
deprecated API

Reviewed-by: chegar

-

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


Draft JEP: 8263693: InetAddress host name and address lookup service provider interface

2021-07-22 Thread Aleksei Efimov

Hi,

For some time now we've been considering adding a standard service 
provider interface for InetAddress APIs that will allow deployment of 
alternative implementations of the underlying host name and address 
resolution mechanisms.


After some prototyping work we have a draft JEP written:
    8263693: InetAddress host name and address lookup service provider 
interface

    https://bugs.openjdk.java.net/browse/JDK-8263693

The prototype implementation (still work in progress) of this JEP can be 
viewed in JDK Sandbox branch:
    git clone https://github.com/openjdk/jdk-sandbox.git -b 
JDK-8244202-nspi-stream-branch


As part of the prototyping work we've also developed two 
proof-of-concept name service providers to demonstrate and verify that 
the provider interface can be used to develop name service 
implementations alternative to the name service implementation shipped 
with the Java platform:

    PoC name service based on JNDI APIs:
https://github.com/openjdk/jdk-sandbox/blob/JDK-8263693-nspi-pocs/JndiBasedPoC/src/main/java/jdk/test/nsp/proof/jndi/ProviderImpl.java
    PoC name service based on Netty library:
https://github.com/openjdk/jdk-sandbox/blob/JDK-8263693-nspi-pocs/NettyBasedPoC/src/main/java/jdk/test/nsp/proof/netty/ProviderImpl.java

Feedback on this JEP before submission would be very welcome.

Best Regards,
Aleksei