RFR: JDK9 message drop interim L10n resource update
ShinyaYoshida
bitterfoxc at gmail.com
Wed Nov 16 04:47:57 UTC 2016
Hi Leo,
Thank you for updating translation.
In jshell, the most becomes better than previous one!
I've noticed incorrect word order in jshell translation:
jshell> /set feedback verbose
| フィードバック・モード: verbose
jshell> int n = 0
n ==> 0
| 次を作成しました: 変数 n : int
jshell> n = 0
n ==> 0
| nに割り当てられました : int
jshell> n
n ==> 0
| nの値: int
jshell> void m(C c) {}
| 次を作成しました: メソッド m(C)まで、参照できません。しかし、 class Cが宣言される
(1): The type name goes far from the variable name.
(2): "まで、参照できません。しかし、 class Cが宣言される" has incorrect order.
They should be following:
jshell> /set feedback verbose
| フィードバック・モード: verbose
jshell> int n = 0
n ==> 0
| 次を作成しました: 変数 n : int
jshell> n = 0
n ==> 0
| n : intに割り当てられました
jshell> n
n ==> 0
| n : intの値
jshell> void m(C c) {}
| 次を作成しました: メソッド m(C)。しかし、 class Cが宣言されるまで、参照できません
Here is fix:
http://cr.openjdk.java.net/~shinyafox/kulla/8169618/l10n_ja.properties.patch.wdiff
This is a wdiff against your udiff at the last 3 lines of
langtools/src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n_ja.properties.
For maintainability, I'd like to suggest breaking lines like
startup.feedback does so
in jdk/internal/jshell/tool/resources/l10n.properties.
Regards,
shinyafox(Shinya Yoshida)
2016-11-14 15:09 GMT+09:00 Leo Jiang <li.jiang at oracle.com>:
> Hi,
>
> Please review:
> Webrev:
> http://cr.openjdk.java.net/~ljiang/8169618/webrev/read/
>
> for bug:
> https://bugs.openjdk.java.net/browse/JDK-8169618
>
> Please help us to review
> - over-translate, e.g. keywords should be not translated
> - not-translate, e.g. some sentences or strings are not translated while
> they should be.
> - any programming syntax error
> - multiple lines property with correct line ending '\n\'
> - the position of placefolder after translation
> - any other issues.
>
> Thanks,
> Leo
>
More information about the jdk9-dev
mailing list