Re: RFR: 8228604: StackMapFrames are missing from redefined class bytes of retransformed classes

2023-01-24 Thread Alex Menkov
On Tue, 24 Jan 2023 00:16:10 GMT, Alex Menkov wrote: > classFileParser drops stack map frames for JDK classes (when verification is > not required). > As a result JvmtiClassFileReconstituter cannot restore the attribute for > class redefinition. > Note that if the class is in CDS archive, the f

Re: RFR: 8299635: Hotspot update for deprecated sprintf in Xcode 14 [v8]

2023-01-24 Thread Mikael Vidstedt
On Thu, 19 Jan 2023 18:54:08 GMT, Xue-Lei Andrew Fan wrote: >> The sprintf is deprecated in Xcode 14 because of security concerns. The >> issue was addressed in >> [JDK-8296812](https://bugs.openjdk.org/browse/JDK-8296812) for hotspot impl, >> and [JDK-8299378](https://bugs.openjdk.org/browse/

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v5]

2023-01-24 Thread Justin Lu
On Tue, 24 Jan 2023 23:56:23 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: RFR: 8228604: StackMapFrames are missing from redefined class bytes of retransformed classes

2023-01-24 Thread David Holmes
On Tue, 24 Jan 2023 00:16:10 GMT, Alex Menkov wrote: > classFileParser drops stack map frames for JDK classes (when verification is > not required). > As a result JvmtiClassFileReconstituter cannot restore the attribute for > class redefinition. > Note that if the class is in CDS archive, the f

Re: RFR: 8228604: StackMapFrames are missing from redefined class bytes of retransformed classes

2023-01-24 Thread Alex Menkov
On Tue, 24 Jan 2023 06:19:51 GMT, David Holmes wrote: >> classFileParser drops stack map frames for JDK classes (when verification is >> not required). >> As a result JvmtiClassFileReconstituter cannot restore the attribute for >> class redefinition. >> Note that if the class is in CDS archive,

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v5]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 23:56:23 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 23:16:02 GMT, Alexey Semenyuk wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change localization codes. Add CNY back > > src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resources/MsiInst

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 22:55:43 GMT, Alexey Semenyuk wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change German help of jar command > > src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resources/MsiInstallerS

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 23:02:58 GMT, Justin Lu wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change localization codes. Add CNY back > > src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins_ja.properties >

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v5]

2023-01-24 Thread Damon Nguyen
> Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision: Revert culture attribute - Changes: - all

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v4]

2023-01-24 Thread Damon Nguyen
> Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision: Remove trailing whitespace - Changes: - a

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Justin Lu
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Naoto Sato
On Tue, 24 Jan 2023 22:40:55 GMT, Justin Lu wrote: >> OK, I can make the manual edit for this > > This change was made because it was compared to the original > messages.properties. In that file, the respective line is given as > `MANAGE_HOTSPOT_MBEANS_IN_COLON_=Manage Hotspot MBeans in:` with

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 22:49:20 GMT, Naoto Sato wrote: >> This change was made because it was compared to the original >> messages.properties. In that file, the respective line is given as >> `MANAGE_HOTSPOT_MBEANS_IN_COLON_=Manage Hotspot MBeans in:` with no trailing >> space. That's why in this

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Joe Wang
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Justin Lu
On Tue, 24 Jan 2023 22:20:22 GMT, Damon Nguyen wrote: >> src/jdk.jconsole/share/classes/sun/tools/jconsole/resources/messages_zh_CN.properties >> line 174: >> >>> 172: >>> MBEANS_TAB_UNSUBSCRIBE_NOTIFICATIONS_BUTTON=\u53D6\u6D88\u8BA2\u9605(&U) >>> 173: >>> MBEANS_TAB_UNSUBSCRIBE_NOTIFICATION

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Phil Race
On Tue, 24 Jan 2023 22:12:26 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 22:14:36 GMT, Naoto Sato wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change localization codes. Add CNY back > > src/jdk.localedata/share/classes/sun/util/resources/ext/CurrencyNames_zh_CN.p

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 22:12:02 GMT, Naoto Sato wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change German help of jar command > > src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resources/WinResources_de.pr

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Damon Nguyen
> Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision: Change localization codes. Add CNY back - C

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 22:05:55 GMT, Naoto Sato wrote: >> Damon Nguyen has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Change German help of jar command > > src/jdk.jconsole/share/classes/sun/tools/jconsole/resources/messages_zh_CN.properti

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Naoto Sato
On Tue, 24 Jan 2023 22:12:26 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v2]

2023-01-24 Thread Damon Nguyen
> Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision: Change German help of jar command - Changes

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update

2023-01-24 Thread Chris Plummer
On Tue, 24 Jan 2023 20:50:00 GMT, Damon Nguyen wrote: > Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed jdk.console and jdk.management.agent changes look good. - Marked as reviewed by cjplummer (Reviewer

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 20:50:00 GMT, Damon Nguyen wrote: > Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed @magicus @LanceAndersen @jonathan-gibbons @pavelrappo @dougxc @lahodaj Could you review the changes from this loc

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/ [v4]

2023-01-24 Thread Johan Sjölen
On Tue, 24 Jan 2023 21:10:31 GMT, Johan Sjölen wrote: >> Do the conversion in the share/jfr/ sub-directory and all of its files. > > Johan Sjölen has updated the pull request incrementally with one additional > commit since the last revision: > > Fix outdated copyright Status: Clang fails bu

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/ [v3]

2023-01-24 Thread Johan Sjölen
On Tue, 24 Jan 2023 20:58:27 GMT, Johan Sjölen wrote: >> Do the conversion in the share/jfr/ sub-directory and all of its files. > > Johan Sjölen has updated the pull request incrementally with one additional > commit since the last revision: > > Manual fix Found one file which now has no ch

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/ [v4]

2023-01-24 Thread Johan Sjölen
> Do the conversion in the share/jfr/ sub-directory and all of its files. Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision: Fix outdated copyright - Changes: - all: https://git.openjdk.org/jdk/pull/12034/files - new: h

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update

2023-01-24 Thread Damon Nguyen
On Tue, 24 Jan 2023 20:50:00 GMT, Damon Nguyen wrote: > Open l10n drop. Files have been updated with translated versions. Whitespace > tool has been ran on files. > All tests passed @prrace @wangweij @naotoj @JoeWang-Java @plummercj @alexeysemenyukoracle Could you review the changes from this l

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/ [v3]

2023-01-24 Thread Johan Sjölen
> Do the conversion in the share/jfr/ sub-directory and all of its files. Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision: Manual fix - Changes: - all: https://git.openjdk.org/jdk/pull/12034/files - new: https://git.o

[jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update

2023-01-24 Thread Damon Nguyen
Open l10n drop. Files have been updated with translated versions. Whitespace tool has been ran on files. All tests passed - Commit messages: - Remove currency symbol entries - Remove whitespace using tool - Remove double quotes generated by translation tool - Revert translation's

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/ [v2]

2023-01-24 Thread Johan Sjölen
> Do the conversion in the share/jfr/ sub-directory and all of its files. Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision: Fix two faulty NULL_STRING misses - Changes: - all: https://git.openjdk.org/jdk/pull/12034/files

Re: RFR: JDK-8300245: Replace NULL with nullptr in share/jfr/

2023-01-24 Thread Johan Sjölen
On Tue, 17 Jan 2023 11:26:12 GMT, Johan Sjölen wrote: > Do the conversion in the share/jfr/ sub-directory and all of its files. Hi @mgronlun, the "fix" comments are there because I intend to revert the macros to regular old `NULL`. I'll copy paste a more informative comment next time :). Also

Re: RFR: 8299635: Hotspot update for deprecated sprintf in Xcode 14 [v8]

2023-01-24 Thread Xue-Lei Andrew Fan
On Fri, 20 Jan 2023 18:20:10 GMT, Xue-Lei Andrew Fan wrote: >> Nothing further from me. I think this is okay as-is now. Thanks. > >> Nothing further from me. I think this is okay as-is now. Thanks. > > @dholmes-ora Thank you for the review! > @XueleiFan You need two reviews before integrating a

Re: RFR: 8300659: Refactor TestMemoryAwareness to use WhiteBox api for host values [v4]

2023-01-24 Thread Matthias Baesken
On Tue, 24 Jan 2023 14:34:35 GMT, Severin Gehwolf wrote: >> Please review this refactoring of a container test. It now uses WhiteBox to >> retrieve the host values it asserts for. In terms of functionality this is >> basically a no-op except for the now more precise assertion on systems with >

Re: RFR: 8300659: Refactor TestMemoryAwareness to use WhiteBox api for host values [v2]

2023-01-24 Thread Severin Gehwolf
On Tue, 24 Jan 2023 08:48:13 GMT, Matthias Baesken wrote: >> Severin Gehwolf 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 one additional >> c

Re: RFR: 8300659: Refactor TestMemoryAwareness to use WhiteBox api for host values [v4]

2023-01-24 Thread Severin Gehwolf
> Please review this refactoring of a container test. It now uses WhiteBox to > retrieve the host values it asserts for. In terms of functionality this is > basically a no-op except for the now more precise assertion on systems with > swap accounting disabled at the kernel level. > > *Testing*

Re: RFR: 8300659: Refactor TestMemoryAwareness to use WhiteBox api for host values [v3]

2023-01-24 Thread Severin Gehwolf
> Please review this refactoring of a container test. It now uses WhiteBox to > retrieve the host values it asserts for. In terms of functionality this is > basically a no-op except for the now more precise assertion on systems with > swap accounting disabled at the kernel level. > > *Testing*

Re: RFR: 8300659: Refactor TestMemoryAwareness to use WhiteBox api for host values [v2]

2023-01-24 Thread Matthias Baesken
On Thu, 19 Jan 2023 17:24:57 GMT, Severin Gehwolf wrote: >> Please review this refactoring of a container test. It now uses WhiteBox to >> retrieve the host values it asserts for. In terms of functionality this is >> basically a no-op except for the now more precise assertion on systems with >