[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-compiler-dev
Subject:    Integrated: 8297879: javadoc link to preview JEP 1000 has grouping character comma
From:       Pavel Rappo <prappo () openjdk ! org>
Date:       2024-03-19 15:25:26
Message-ID: 7PArJXG1ZKRHbwf2SBnuB6z1YIOC9iYcMgHNX_MHmSI=.aa8998d0-08ec-44de-88e1-781df1a695ae () github ! com
[Download RAW message or body]

On Mon, 18 Mar 2024 14:53:44 GMT, Pavel Rappo <prappo@openjdk.org> wrote:

> Please review this simple bugfix to properly construct links to preview JEPs.
> 
> The most straightforward fix I could think of was to pass `String` rather than \
> `int` (`Integer`) to a method, which eventually calls \
> `java.text.MessageFormat.format(String, Object...)`. 
> For the test, I decided to be ~lazy~ practical and piggyback on the existing \
> infrastructure. The alternatives were: 
> 1. slap `noreg-hard` on the JBS bug and skip testing
> 2. create a sophisticated test that dynamically adds a constant into the \
> `PreviewFeature.Feature` enum, annotates some class with `PreviewFeature` with that \
> constant, and finally documents that class with `PreviewFeature` patched into \
> `java.base` 
> While (1) is insufficient, (2) seems overkill in this case.

This pull request has now been integrated.

Changeset: f140eb4c
Author:    Pavel Rappo <prappo@openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f140eb4c3a47e1479d62fe1eef16bbbea92892bc
                
Stats:     5 lines in 3 files changed: 0 ins; 0 del; 5 mod

8297879: javadoc link to preview JEP 1000 has grouping character comma

Reviewed-by: jjg, vromero

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

PR: https://git.openjdk.org/jdk/pull/18350


[prev in list] [next in list] [prev in thread] [next in thread]