Re: RFR: 8289562: Change bugs.java.com and bugreport.java.com URL's to https

2022-08-01 Thread Phil Race
On Mon, 11 Jul 2022 08:04:20 GMT, Joe wrote: > 8289562: Change bugs.java.com and bugreport.java.com URL's to https Looks fine to me. However the langtools / javadoc folks seem like they should be primary reviewers since it is mostly messages from the tools. - Marked as reviewed by

Re: [jdk19] RFR: 8290889: JDK 19 RDP2 L10n resource files update - msgdrop 10 [v4]

2022-08-01 Thread Naoto Sato
On Fri, 29 Jul 2022 18:13:25 GMT, Alisen Chung wrote: >> open l10n msg drop >> All tests passed. > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > added comments in CurrencyNames root in base, moved US CurrencyNames back > t

Re: RFR: 8289562: Change bugs.java.com and bugreport.java.com URL's to https

2022-08-01 Thread Fairoz Matte
On Mon, 11 Jul 2022 08:04:20 GMT, Joe wrote: > 8289562: Change bugs.java.com and bugreport.java.com URL's to https It is a trivial change and looks good to me (I am not a reviewer). - PR: https://git.openjdk.org/jdk/pull/9445

RFR: 8289562: Change bugs.java.com and bugreport.java.com URL's to https

2022-08-01 Thread Joe
8289562: Change bugs.java.com and bugreport.java.com URL's to https - Commit messages: - Merge branch 'openjdk:master' into master - changed the hrl from http to https JDK-8289562 Changes: https://git.openjdk.org/jdk/pull/9445/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr

Re: RFR: 8289658: Avoid redundant LinkedHashMap.get call in TagletManager.addNewSimpleCustomTag [v4]

2022-08-01 Thread Andrey Turbanov
> Instead of pair `LinkedHashMap.get`+`LinkedHashMap.remove` calls, we can use > value returned from single `allTaglets.remove` call. > It's shorter and a bit faster. > https://github.com/openjdk/jdk/blob/d46f404b3179c66e8e5775a9e2253c95238153c7/src/jdk.javadoc/share/classes/jdk/javadoc/internal/d