On Thu, 5 Dec 2024 22:36:12 GMT, Justin Lu <j...@openjdk.org> wrote:

> Please review this PR which contains the open L10n drop changes for RDP1.
> 
> I recommend viewing the improved diffs which are built out by Jon's tool 
> here: https://cr.openjdk.org/~jlu/output/.  As always, I can not confirm the 
> correctness on the quality of the translations themselves.
> 
> Generally speaking, observed differences in punctuation can be attributed to 
> language rules. Any keys requiring localization that came in after the JDK 
> was submitted for translations will be handled in the RDP2 L10n drop.

I don't speak any of the languages changed here, but I have looked at:
- the new concise helps for the `java` launcher, added some comments mostly on 
alignment
- in the `javac` message `transitive modifier for java.base`, `transitive` is a 
literal modifier appearing in the source, so possibly it should not be 
translated (we can enhance the English bundle to say that). There's also a 
comment inline on this.
- for JShell bundles, I note there's no update for Japanese - that's OK, just 
making sure its intentional. There was also a relatively late change to the 
JShell bundles:
https://github.com/openjdk/jdk/commit/bcebb0c53c1e4629ebde534e237a86c161130fff#diff-fd30aa079239d0b535a6eb014ee0bc8d16709969c87cc9f6c652f91d75aec769
I assume that change was too late for this drop.

Overall, I think the parts I checked are OK (to the degree I can understand 
it), with possible (future) improvements for consideration.

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

> 42: 
> 43: # Translators please note do not translate the options themselves
> 44: java.launcher.opt.concise.header  =   Verwendung: java [Java-Optionen...] 
> <Anwendung> [Anwendungsargumente...]\n\nDabei ist <Anwendung> einer der 
> folgenden Werte:\n  <Hauptklasse>                Zum Ausführen der 
> Hauptmethode einer kompilierten Hauptklasse\n  -jar <JAR-Datei>.jar         
> Zum Ausführen der Hauptklasse eines JAR-Archivs\n  -m <Modul>[/<Hauptklasse>] 
> Zum Ausführen der Hauptklasse eines Moduls\n  <Quelldatei>.java          Zum 
> Kompilieren und Ausführen eines Quelldateiprogramms\n\nDabei sind die 
> folgenden wichtigen Java-Optionen verfügbar:\n  --class-path <Classpath>\n    
> <Classpath> ist eine durch "{0}" getrennte Liste der Verzeichnisse und 
> JAR-Archive, in denen nach Klassendateien gesucht werden soll\n  
> --module-path <Modulpfad>\n    <Modulpfad> ist eine durch "{0}" getrennte 
> Liste der Verzeichnisse und JAR-Archive, in denen nach Modulen gesucht werden 
> soll\n  -version\n    Zum Ausgeben der Produktversion in den Fehlerstream und 
> Beenden des Vorgangs\n\nFür 
 weitere Verwendungshilfe:           java --help\nFür eine interaktive 
Java-Umgebung:    jshell

This gives the following output:

Verwendung: java [Java-Optionen...] <Anwendung> [Anwendungsargumente...]

Dabei ist <Anwendung> einer der folgenden Werte:
  <Hauptklasse>                Zum Ausführen der Hauptmethode einer 
kompilierten Hauptklasse
  -jar <JAR-Datei>.jar         Zum Ausführen der Hauptklasse eines JAR-Archivs
  -m <Modul>[/<Hauptklasse>] Zum Ausführen der Hauptklasse eines Moduls
  <Quelldatei>.java          Zum Kompilieren und Ausführen eines 
Quelldateiprogramms

Dabei sind die folgenden wichtigen Java-Optionen verfügbar:
  --class-path <Classpath>
    <Classpath> ist eine durch ":" getrennte Liste der Verzeichnisse und 
JAR-Archive, in denen nach Klassendateien gesucht werden soll
  --module-path <Modulpfad>
    <Modulpfad> ist eine durch ":" getrennte Liste der Verzeichnisse und 
JAR-Archive, in denen nach Modulen gesucht werden soll
  -version
    Zum Ausgeben der Produktversion in den Fehlerstream und Beenden des Vorgangs

Für weitere Verwendungshilfe:           java --help
Für eine interaktive Java-Umgebung:    jshell


It would be nicer if the `Zum ...` texts would be aligned, and the `java` and 
`jshell` at the end would be aligned. I.e.:


Verwendung: java [Java-Optionen...] <Anwendung> [Anwendungsargumente...]

Dabei ist <Anwendung> einer der folgenden Werte:
  <Hauptklasse>                Zum Ausführen der Hauptmethode einer 
kompilierten Hauptklasse
  -jar <JAR-Datei>.jar         Zum Ausführen der Hauptklasse eines JAR-Archivs
  -m <Modul>[/<Hauptklasse>]   Zum Ausführen der Hauptklasse eines Moduls
  <Quelldatei>.java            Zum Kompilieren und Ausführen eines 
Quelldateiprogramms

Dabei sind die folgenden wichtigen Java-Optionen verfügbar:
  --class-path <Classpath>
    <Classpath> ist eine durch ":" getrennte Liste der Verzeichnisse und 
JAR-Archive, in denen nach Klassendateien gesucht werden soll
  --module-path <Modulpfad>
    <Modulpfad> ist eine durch ":" getrennte Liste der Verzeichnisse und 
JAR-Archive, in denen nach Modulen gesucht werden soll
  -version
    Zum Ausgeben der Produktversion in den Fehlerstream und Beenden des Vorgangs

Für weitere Verwendungshilfe:          java --help
Für eine interaktive Java-Umgebung:    jshell

src/java.base/share/classes/sun/launcher/resources/launcher_ja.properties line 
46:

> 44: 
> 45: # Translators please note do not translate the options themselves
> 46: java.launcher.opt.concise.header  =   使用方法: java [java options...] 
> <application> [application arguments...]\n\n<application>は次のいずれかです:\n  
> <mainclass>                コンパイルされたメイン・クラスのメイン・メソッドを実行します\n  -jar 
> <jarfile>.jar         JARアーカイブのメイン・クラスを実行します\n  -m <module>[/<mainclass>]  
> モジュールのメイン・クラスを実行します\n  <sourcefile>.java          
> ソースファイル・プログラムをコンパイルおよび実行します\n\n主要なjavaオプションには次が含まれます:\n  --class-path <class 
> path>\n    <class 
> path>は、クラス・ファイルを検索するためのディレクトリおよびJARアーカイブのリストであり、"{0}"で区切られます\n  --module-path 
> <module path>\n    <module path>は、モジュールを検索するためのディレクトリおよびJARアーカイブのリストであり、
 "{0}"で区切られます\n  -version\n    
製品バージョンをエラー・ストリームに出力し終了します\n\n使用方法についての追加のヘルプの場合:           java 
--help\n対話型のJava環境の場合:    jshell

Similarly to the German version, the currently of `java` and `jshell` are not 
aligned:

使用方法: java [java options...] <application> [application arguments...]

<application>は次のいずれかです:
  <mainclass>                コンパイルされたメイン・クラスのメイン・メソッドを実行します
  -jar <jarfile>.jar         JARアーカイブのメイン・クラスを実行します
  -m <module>[/<mainclass>]  モジュールのメイン・クラスを実行します
  <sourcefile>.java          ソースファイル・プログラムをコンパイルおよび実行します

主要なjavaオプションには次が含まれます:
  --class-path <class path>
    <class path>は、クラス・ファイルを検索するためのディレクトリおよびJARアーカイブのリストであり、":"で区切られます
  --module-path <module path>
    <module path>は、モジュールを検索するためのディレクトリおよびJARアーカイブのリストであり、":"で区切られます
  -version
    製品バージョンをエラー・ストリームに出力し終了します

使用方法についての追加のヘルプの場合:           java --help
対話型のJava環境の場合:    jshell


It would be nicer if they did (assuming it is doable):

使用方法: java [java options...] <application> [application arguments...]

<application>は次のいずれかです:
  <mainclass>                コンパイルされたメイン・クラスのメイン・メソッドを実行します
  -jar <jarfile>.jar         JARアーカイブのメイン・クラスを実行します
  -m <module>[/<mainclass>]  モジュールのメイン・クラスを実行します
  <sourcefile>.java          ソースファイル・プログラムをコンパイルおよび実行します

主要なjavaオプションには次が含まれます:
  --class-path <class path>
    <class path>は、クラス・ファイルを検索するためのディレクトリおよびJARアーカイブのリストであり、":"で区切られます
  --module-path <module path>
    <module path>は、モジュールを検索するためのディレクトリおよびJARアーカイブのリストであり、":"で区切られます
  -version
    製品バージョンをエラー・ストリームに出力し終了します

使用方法についての追加のヘルプの場合:  java --help
対話型のJava環境の場合:             jshell

src/java.base/share/classes/sun/launcher/resources/launcher_zh_CN.properties 
line 44:

> 42: 
> 43: # Translators please note do not translate the options themselves
> 44: java.launcher.opt.concise.header  =   用法:java [java options...] 
> <application> [application arguments...]\n\n其中,<application> 为以下项之一:\n  
> <mainclass>                执行编译的主类的 main 方法\n  -jar <jarfile>.jar         执行 
> JAR 档案的主类\n  -m <module>[/<mainclass>]  执行模块的主类\n  <sourcefile>.java          
> 编译和执行源文件程序\n\n其中,主要的 java 选项包括:\n  --class-path <class path>\n    其中,<class 
> path> 是用于搜索类文件的目录和 JAR 档案列表(以 "{0}" 分隔)\n  --module-path <module path>\n    
> 其中,<module path> 是用于搜索模块的目录和 JAR 档案列表(以 "{0}" 分隔)\n  -version\n    
> 将产品版本输出到错误流并退出\n\n获取有关用法的其他帮助:           java --help\n获取交互式 Java 环境:    jshell

It would be nicer if the  `java` and `jshell` could be aligned in the last 
section, but maybe it is not possible?

用法:java [java options...] <application> [application arguments...]

其中,<application> 为以下项之一:
  <mainclass>                执行编译的主类的 main 方法
  -jar <jarfile>.jar         执行 JAR 档案的主类
  -m <module>[/<mainclass>]  执行模块的主类
  <sourcefile>.java          编译和执行源文件程序

其中,主要的 java 选项包括:
  --class-path <class path>
    其中,<class path> 是用于搜索类文件的目录和 JAR 档案列表(以 ":" 分隔)
  --module-path <module path>
    其中,<module path> 是用于搜索模块的目录和 JAR 档案列表(以 ":" 分隔)
  -version
    将产品版本输出到错误流并退出

获取有关用法的其他帮助:           java --help
获取交互式 Java 环境:    jshell

src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler_de.properties
 line 2302:

> 2300: compiler.misc.feature.module.imports=Modulimporte
> 2301: 
> 2302: compiler.misc.feature.java.base.transitive=Transitiver Modifikator für 
> java.base

Here (and in other languages as well), maybe the `transitive` should remain 
intact? `transitive` is what literally appears in the source code - similar to 
e.g. `static` or `private`.

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

Marked as reviewed by jlahoda (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22588#pullrequestreview-2491653435
PR Review Comment: https://git.openjdk.org/jdk/pull/22588#discussion_r1877841057
PR Review Comment: https://git.openjdk.org/jdk/pull/22588#discussion_r1877847825
PR Review Comment: https://git.openjdk.org/jdk/pull/22588#discussion_r1877854317
PR Review Comment: https://git.openjdk.org/jdk/pull/22588#discussion_r1877712280

Reply via email to