Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Damon Nguyen
On Fri, 16 Dec 2022 17:26:18 GMT, Phil Race  wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Revert old translation. Fix lang codes
>
> src/demo/share/jfc/SwingSet2/resources/swingset_zh_CN.properties line 430:
> 
>> 428: 
>> ProgressBarDemo.accessible_text_area_name=\u52A0\u8F7D\u7684\u6587\u672C\u6B63\u5728\u9010\u6E10\u589E\u591A
>> 429: 
>> 430: ProgressBarDemo.accessible_text_area_description=\u8FD9\u4E2A JTextArea 
>> \u7531\u6765\u81EA\u7F13\u51B2\u533A\u7684\u6587\u672C\u9010\u4E2A\u5B57\u7B26\u5730\u586B\u5145\uFF0C\u540C\u65F6\u7A97\u53E3\u5E95\u90E8\u7684\u8FDB\u5EA6\u680F\u5C06\u663E\u793A\u52A0\u8F7D\u7684\u8FDB\u5EA6
> 
> I was asked to review SwingSet2 changes but this isn't code.
> But apparently I am not the right person to review this since I cannot 
> understand the changes (not a DE, CN, or JA language expert) nor the reason 
> for them. If this is a change to a unicode comma (as implied in an off-review 
> message), I have no idea why .. it seems to be just asking for trouble.

Thanks. I removed the changes to swingset's translation to avoid causing any 
issues.

-

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Damon Nguyen
On Fri, 16 Dec 2022 09:46:14 GMT, Daniel Fuchs  wrote:

>> src/java.base/share/classes/sun/launcher/resources/launcher_de.properties 
>> line 34:
>> 
>>> 32: # Translators please note do not translate the options themselves
>>> 33: java.launcher.opt.footer = \-cp >> und ZIP-/JAR-Dateien>\n-classpath >> und ZIP-/JAR-Dateien>\n--class-path >> und ZIP-/JAR-Dateien>\n  Eine durch {0} getrennte Liste mit 
>>> Verzeichnissen, JAR-Archiven\n  und ZIP-Archiven, in denen 
>>> nach Klassendateien gesucht wird.\n-p \n--module-path 
>>> ...\n  Eine durch {0} getrennte Liste mit 
>>> Verzeichnissen, von denen jedes Verzeichnis\n  ein 
>>> Verzeichnis mit Modulen ist.\n--upgrade-module-path ...\n
>>>   Eine durch {0} getrennte Liste mit Verzeichnissen, von denen 
>>> jedes Verzeichnis\n  ein Verzeichnis mit Modulen ist, die 
>>> upgradef\u00E4hige\n  Module im Laufzeitimage ersetzen\n
>>> --add-modules [,...]\n  Root-Module, 
>>> die zus\u00E4tzlich zum anf\u00E
 4nglichen Modul aufgel\u00F6st werden sollen.\n   
kann auch wie folgt lauten: ALL-DEFAULT, ALL-SYSTEM,\n  
ALL-MODULE-PATH.\n--enable-native-access [,...]\n 
 Module, die eingeschr\u00E4nkte native Vorg\u00E4nge 
ausf\u00FChren d\u00FCrfen.\n   kann auch 
ALL-UNNAMED lauten.\n--list-modules\n  Listet beobachtbare 
Module auf und beendet den Vorgang\n-d \n--describe-module 
\n  Beschreibt ein Modul und beendet den Vorgang\n   
 --dry-run Erstellt eine VM und l\u00E4dt die Hauptklasse, f\u00FChrt aber 
nicht die Hauptmethode aus.\n  Die Option "--dry-run" kann 
n\u00FCtzlich sein, um die\n  Befehlszeilenoptionen, wie die 
Modulsystemkonfiguration, zu validieren.\n--validate-modules\n  
Validiert alle Module und beendet den Vorgang\n  Die Option 
"--va
 lidate-modules" kann n\u00FCtzlich sein, um\n  Konflikte und 
andere Fehler mit Modulen auf dem Modulpfad zu ermitteln.\n
-D=\n  Legt eine Systemeigenschaft fest\n
-verbose:[class|module|gc|jni]\n  Aktiviert die Verbose-Ausgabe 
f\u00FCr das angegebene Subsystem\n-version  Gibt die Produktversion an 
den Fehlerstream aus und beendet den Vorgang\n--version  Gibt die 
Produktversion an den Outputstream aus und beendet den Vorgang\n
-showversion  Gibt die Produktversion an den Fehlerstream aus und setzt den 
Vorgang fort\n--show-version\n  Gibt die Produktversion an 
den Outputstream aus und setzt den Vorgang fort\n--show-module-resolution\n 
 Zeigt die Modulaufl\u00F6sungsausgabe beim Start an\n-? -h 
-help\n  Gibt diese Hilfemeldung an den Fehlerstream aus\n
--helpGibt diese Hilfemeldung an den Outputstream aus\n-X   
 Gibt Hil
 fe zu zus\u00E4tzlichen Optionen an den Fehlerstream aus\n--help-extra  
Gibt Hilfe zu zus\u00E4tzlichen Optionen an den Outputstream aus\n
-ea[:...|:]\n
-enableassertions[:...|:]\n  Aktiviert 
Assertions mit angegebener Granularit\u00E4t\n
-da[:...|:]\n
-disableassertions[:...|:]\n  
Deaktiviert Assertions mit angegebener Granularit\u00E4t\n-esa | 
-enablesystemassertions\n  Aktiviert System-Assertions\n
-dsa | -disablesystemassertions\n  Deaktiviert 
System-Assertions\n-agentlib:[=]\n  
L\u00E4dt die native Agent Library . Beispiel: -agentlib:jdwp\n
  siehe auch -agentlib:jdwp=help\n
-agentpath:[=]\n  L\u00E4dt die native Agent 
Library mit dem vollst\u00E4ndigen Pfadnamen\n
-javaagent:[=]
 \n  \
>>> 34: L\u00E4dt den Java-Programmiersprachen-Agent, siehe 
>>> java.lang.instrument\n-splash:\n  Zeigt den 
>>> Startbildschirm mit einem angegebenen Bild an\n  Skalierte 
>>> HiDPI-Bilder werden automatisch unterst\u00FCtzt und verwendet,\n   
>>>falls verf\u00FCgbar. Der nicht skalierte Bilddateiname (Beispiel: 
>>> image.ext)\n  muss immer als Argument an die Option 
>>> "-splash" \u00FCbergeben werden.\n  Das am besten geeignete 
>>> angegebene skalierte Bild wird\n  automatisch 
>>> ausgew\u00E4hlt.\n  Weitere Informationen finden Sie in der 
>>> Dokumentation zur SplashScreen-API\n@argument files\n  
>>> Eine oder mehrere Argumentdateien mit Optionen\n-disable-@files\n   
>>>Verhindert die weitere Erweiterung von Argumentdateien\n
>>> --enable-preview\n  L\u00E4sst zu, das Klassen von 
>>> Vorschaufeatures dieses Release abh\u00E4ngig sind\nUm ein Argument
  f\

Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Damon Nguyen
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

Right, Naoto's comment is correct. This time, the source extraction used some 
older files. To resolve this for this drop, I went through the files and 
reverted any changes that would be deemed as regressions or outdated files. Any 
other updates missed in this drop will be handled in the close upcoming drop in 
January.

-

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Naoto Sato
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

There is another translation drop scheduled in January, so our plan is to 
figure out why the source extraction in this drop mistakenly used old files 
till then, then address it in the January l10n drop.

-

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Alan Bateman
On Fri, 16 Dec 2022 03:55:29 GMT, Damon Nguyen  wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Revert old translation. Fix lang codes
>
> @dfuch Could I get a review from you as well? I believe this translation 
> covers some of the files for your team.

@DamonGuy There seems to be several glitches in this refresh. It's also 
surprising that the translations appear to be done on several out of date 
files. On the positive front, it's good to see this update has caught the 
attention of Naoto, Weijun, and others. Once the identified issues have been 
fixed then it would be good to try to get a review from someone that is native 
in each of the languages.

-

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Daniel Fuchs
On Fri, 16 Dec 2022 09:20:07 GMT, Daniel Fuchs  wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Revert old translation. Fix lang codes
>
> src/java.base/share/classes/sun/launcher/resources/launcher_de.properties 
> line 34:
> 
>> 32: # Translators please note do not translate the options themselves
>> 33: java.launcher.opt.footer = \-cp > und ZIP-/JAR-Dateien>\n-classpath > und ZIP-/JAR-Dateien>\n--class-path > und ZIP-/JAR-Dateien>\n  Eine durch {0} getrennte Liste mit 
>> Verzeichnissen, JAR-Archiven\n  und ZIP-Archiven, in denen 
>> nach Klassendateien gesucht wird.\n-p \n--module-path 
>> ...\n  Eine durch {0} getrennte Liste mit 
>> Verzeichnissen, von denen jedes Verzeichnis\n  ein 
>> Verzeichnis mit Modulen ist.\n--upgrade-module-path ...\n 
>>  Eine durch {0} getrennte Liste mit Verzeichnissen, von denen 
>> jedes Verzeichnis\n  ein Verzeichnis mit Modulen ist, die 
>> upgradef\u00E4hige\n  Module im Laufzeitimage ersetzen\n
>> --add-modules [,...]\n  Root-Module, 
>> die zus\u00E4tzlich zum anf\u00E4
 nglichen Modul aufgel\u00F6st werden sollen.\n   
kann auch wie folgt lauten: ALL-DEFAULT, ALL-SYSTEM,\n  
ALL-MODULE-PATH.\n--enable-native-access [,...]\n 
 Module, die eingeschr\u00E4nkte native Vorg\u00E4nge 
ausf\u00FChren d\u00FCrfen.\n   kann auch 
ALL-UNNAMED lauten.\n--list-modules\n  Listet beobachtbare 
Module auf und beendet den Vorgang\n-d \n--describe-module 
\n  Beschreibt ein Modul und beendet den Vorgang\n   
 --dry-run Erstellt eine VM und l\u00E4dt die Hauptklasse, f\u00FChrt aber 
nicht die Hauptmethode aus.\n  Die Option "--dry-run" kann 
n\u00FCtzlich sein, um die\n  Befehlszeilenoptionen, wie die 
Modulsystemkonfiguration, zu validieren.\n--validate-modules\n  
Validiert alle Module und beendet den Vorgang\n  Die Option 
"--val
 idate-modules" kann n\u00FCtzlich sein, um\n  Konflikte und 
andere Fehler mit Modulen auf dem Modulpfad zu ermitteln.\n
-D=\n  Legt eine Systemeigenschaft fest\n
-verbose:[class|module|gc|jni]\n  Aktiviert die Verbose-Ausgabe 
f\u00FCr das angegebene Subsystem\n-version  Gibt die Produktversion an 
den Fehlerstream aus und beendet den Vorgang\n--version  Gibt die 
Produktversion an den Outputstream aus und beendet den Vorgang\n
-showversion  Gibt die Produktversion an den Fehlerstream aus und setzt den 
Vorgang fort\n--show-version\n  Gibt die Produktversion an 
den Outputstream aus und setzt den Vorgang fort\n--show-module-resolution\n 
 Zeigt die Modulaufl\u00F6sungsausgabe beim Start an\n-? -h 
-help\n  Gibt diese Hilfemeldung an den Fehlerstream aus\n
--helpGibt diese Hilfemeldung an den Outputstream aus\n-X   
 Gibt Hilf
 e zu zus\u00E4tzlichen Optionen an den Fehlerstream aus\n--help-extra  
Gibt Hilfe zu zus\u00E4tzlichen Optionen an den Outputstream aus\n
-ea[:...|:]\n
-enableassertions[:...|:]\n  Aktiviert 
Assertions mit angegebener Granularit\u00E4t\n
-da[:...|:]\n
-disableassertions[:...|:]\n  
Deaktiviert Assertions mit angegebener Granularit\u00E4t\n-esa | 
-enablesystemassertions\n  Aktiviert System-Assertions\n
-dsa | -disablesystemassertions\n  Deaktiviert 
System-Assertions\n-agentlib:[=]\n  
L\u00E4dt die native Agent Library . Beispiel: -agentlib:jdwp\n
  siehe auch -agentlib:jdwp=help\n
-agentpath:[=]\n  L\u00E4dt die native Agent 
Library mit dem vollst\u00E4ndigen Pfadnamen\n
-javaagent:[=]\
 n  \
>> 34: L\u00E4dt den Java-Programmiersprachen-Agent, siehe 
>> java.lang.instrument\n-splash:\n  Zeigt den 
>> Startbildschirm mit einem angegebenen Bild an\n  Skalierte 
>> HiDPI-Bilder werden automatisch unterst\u00FCtzt und verwendet,\n
>>   falls verf\u00FCgbar. Der nicht skalierte Bilddateiname (Beispiel: 
>> image.ext)\n  muss immer als Argument an die Option 
>> "-splash" \u00FCbergeben werden.\n  Das am besten geeignete 
>> angegebene skalierte Bild wird\n  automatisch 
>> ausgew\u00E4hlt.\n  Weitere Informationen finden Sie in der 
>> Dokumentation zur SplashScreen-API\n@argument files\n  
>> Eine oder mehrere Argumentdateien mit Optionen\n-disable-@files\n
>>   Verhindert die weitere Erweiterung von Argumentdateien\n
>> --enable-p

Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-16 Thread Daniel Fuchs
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

src/java.base/share/classes/sun/launcher/resources/launcher_de.properties line 
34:

> 32: # Translators please note do not translate the options themselves
> 33: java.launcher.opt.footer = \-cp  und ZIP-/JAR-Dateien>\n-classpath  ZIP-/JAR-Dateien>\n--class-path  ZIP-/JAR-Dateien>\n  Eine durch {0} getrennte Liste mit 
> Verzeichnissen, JAR-Archiven\n  und ZIP-Archiven, in denen 
> nach Klassendateien gesucht wird.\n-p \n--module-path 
> ...\n  Eine durch {0} getrennte Liste mit 
> Verzeichnissen, von denen jedes Verzeichnis\n  ein 
> Verzeichnis mit Modulen ist.\n--upgrade-module-path ...\n  
> Eine durch {0} getrennte Liste mit Verzeichnissen, von denen 
> jedes Verzeichnis\n  ein Verzeichnis mit Modulen ist, die 
> upgradef\u00E4hige\n  Module im Laufzeitimage ersetzen\n
> --add-modules [,...]\n  Root-Module, 
> die zus\u00E4tzlich zum anf\u00E4n
 glichen Modul aufgel\u00F6st werden sollen.\n   
kann auch wie folgt lauten: ALL-DEFAULT, ALL-SYSTEM,\n  
ALL-MODULE-PATH.\n--enable-native-access [,...]\n 
 Module, die eingeschr\u00E4nkte native Vorg\u00E4nge 
ausf\u00FChren d\u00FCrfen.\n   kann auch 
ALL-UNNAMED lauten.\n--list-modules\n  Listet beobachtbare 
Module auf und beendet den Vorgang\n-d \n--describe-module 
\n  Beschreibt ein Modul und beendet den Vorgang\n   
 --dry-run Erstellt eine VM und l\u00E4dt die Hauptklasse, f\u00FChrt aber 
nicht die Hauptmethode aus.\n  Die Option "--dry-run" kann 
n\u00FCtzlich sein, um die\n  Befehlszeilenoptionen, wie die 
Modulsystemkonfiguration, zu validieren.\n--validate-modules\n  
Validiert alle Module und beendet den Vorgang\n  Die Option 
"--vali
 date-modules" kann n\u00FCtzlich sein, um\n  Konflikte und 
andere Fehler mit Modulen auf dem Modulpfad zu ermitteln.\n
-D=\n  Legt eine Systemeigenschaft fest\n
-verbose:[class|module|gc|jni]\n  Aktiviert die Verbose-Ausgabe 
f\u00FCr das angegebene Subsystem\n-version  Gibt die Produktversion an 
den Fehlerstream aus und beendet den Vorgang\n--version  Gibt die 
Produktversion an den Outputstream aus und beendet den Vorgang\n
-showversion  Gibt die Produktversion an den Fehlerstream aus und setzt den 
Vorgang fort\n--show-version\n  Gibt die Produktversion an 
den Outputstream aus und setzt den Vorgang fort\n--show-module-resolution\n 
 Zeigt die Modulaufl\u00F6sungsausgabe beim Start an\n-? -h 
-help\n  Gibt diese Hilfemeldung an den Fehlerstream aus\n
--helpGibt diese Hilfemeldung an den Outputstream aus\n-X   
 Gibt Hilfe
  zu zus\u00E4tzlichen Optionen an den Fehlerstream aus\n--help-extra  Gibt 
Hilfe zu zus\u00E4tzlichen Optionen an den Outputstream aus\n
-ea[:...|:]\n
-enableassertions[:...|:]\n  Aktiviert 
Assertions mit angegebener Granularit\u00E4t\n
-da[:...|:]\n
-disableassertions[:...|:]\n  
Deaktiviert Assertions mit angegebener Granularit\u00E4t\n-esa | 
-enablesystemassertions\n  Aktiviert System-Assertions\n
-dsa | -disablesystemassertions\n  Deaktiviert 
System-Assertions\n-agentlib:[=]\n  
L\u00E4dt die native Agent Library . Beispiel: -agentlib:jdwp\n
  siehe auch -agentlib:jdwp=help\n
-agentpath:[=]\n  L\u00E4dt die native Agent 
Library mit dem vollst\u00E4ndigen Pfadnamen\n
-javaagent:[=]\n
   \
> 34: L\u00E4dt den Java-Programmiersprachen-Agent, siehe 
> java.lang.instrument\n-splash:\n  Zeigt den 
> Startbildschirm mit einem angegebenen Bild an\n  Skalierte 
> HiDPI-Bilder werden automatisch unterst\u00FCtzt und verwendet,\n 
>  falls verf\u00FCgbar. Der nicht skalierte Bilddateiname (Beispiel: 
> image.ext)\n  muss immer als Argument an die Option "-splash" 
> \u00FCbergeben werden.\n  Das am besten geeignete angegebene 
> skalierte Bild wird\n  automatisch ausgew\u00E4hlt.\n 
>  Weitere Informationen finden Sie in der Dokumentation zur 
> SplashScreen-API\n@argument files\n  Eine oder mehrere 
> Argumentdateien mit Optionen\n-disable-@files\n  
> Verhindert die weitere Erweiterung von Argumentdateien\n
> --enable-preview\

Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-15 Thread Alexander Matveev
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

jpackage changes looks good.

-

Marked as reviewed by almatvee (Reviewer).

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-15 Thread Chris Plummer
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

`src/jdk.jdi/share/classes/com/sun/tools` changes look good.

-

Marked as reviewed by cjplummer (Reviewer).

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-15 Thread Damon Nguyen
On Fri, 16 Dec 2022 03:38:54 GMT, Damon Nguyen  wrote:

>> Open l10n drop
>> All tests passed
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Revert old translation. Fix lang codes

@dfuch Could I get a review from you as well? I believe this translation covers 
some of the files for your team.

-

PR: https://git.openjdk.org/jdk20/pull/35


Re: [jdk20] RFR: 8298133: JDK 20 RDP1 L10n resource files update - msgdrop 10 [v5]

2022-12-15 Thread Damon Nguyen
> Open l10n drop
> All tests passed

Damon Nguyen has updated the pull request incrementally with one additional 
commit since the last revision:

  Revert old translation. Fix lang codes

-

Changes:
  - all: https://git.openjdk.org/jdk20/pull/35/files
  - new: https://git.openjdk.org/jdk20/pull/35/files/57c42206..29880af9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk20&pr=35&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk20&pr=35&range=03-04

  Stats: 18 lines in 9 files changed: 6 ins; 0 del; 12 mod
  Patch: https://git.openjdk.org/jdk20/pull/35.diff
  Fetch: git fetch https://git.openjdk.org/jdk20 pull/35/head:pull/35

PR: https://git.openjdk.org/jdk20/pull/35