On Sun, 9 Jun 2024 22:49:11 GMT, Justin Lu <j...@openjdk.org> wrote:

>> This issue is responsible for updating the translations of all the 
>> localize(able) resources in the JDK. Primarily, the changes between JDK 22 
>> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>> 
>> The translation tool adjusted some definitions, which causes some changes in 
>> localized files where the source file had no changes. This causes some words 
>> being reverted from localized languages to English, and some had its 
>> definitions changed.
>> 
>> Alternatively, the diffs are viewable here and was generated using John 
>> Gibbon's diff tool for l10n:
>> https://cr.openjdk.org/~dnguyen/output2/
>
> src/java.desktop/share/classes/sun/print/resources/serviceui_de.properties 
> line 1:
> 
>> 1: #
> 
> It looks like these .properties files need to have their copyright years 
> bumped. I believe since the original was not bumped, the translation tool did 
> not reflect the correct year in the localized versions.

Right, the source file has the copyright year of 2019, and these translated 
files reflect that, so I believe this is OK.

> src/java.desktop/share/classes/sun/print/resources/serviceui_zh_CN.properties 
> line 66:
> 
>> 64: label.size=大小(&Z):
>> 65: label.source=来源(&C):
>> 66: label.outputbins=输出托盘(&P):
> 
> Looks like the translation tool changed this from a colon (U+003A) to a full 
> width colon (U+FF1A). There are l10n rules when it comes to punctuation, I am 
> not sure if this is a result of that. It looks like the surrounding 
> key/values are simply using (U+003A) colons. If we decide to revert this 
> colon change, we need to also probably file a bug against the translation 
> tool.

Right, I agree. I'm OK with either decision too, but I'm leaning towards 
reverting this to standardize the colons since the rest are (U+003A). Maybe a 
reviewer from desktop can chime in once this PR is undrafted.

> src/java.xml/share/classes/com/sun/org/apache/xerces/internal/impl/msg/XMLMessages_de.properties
>  line 278:
> 
>> 276:         UndeclaredElementInContentSpec = Contentmodell des Elements 
>> "{0}" verweist auf das nicht deklarierte Element "{1}".
>> 277:         UniqueNotationName = Deklaration für die Notation "{0}" ist 
>> nicht eindeutig. Ein jeweiliger Name darf nicht in mehreren 
>> Notationsdeklarationen deklariert werden.
>> 278:         ENTITYFailedInitializeGrammar =  ENTITYDatatype-Validator: 
>> Nicht erfolgreich. Initialisierungsmethode muss mit einer gültigen 
>> Grammatikreferenz aufgerufen werden. \t
> 
> It looks like the _zh_CN_ file should also have the `\t` removed, but such 
> changes are done by the translation tool. I think in this case, we should 
> manually remove it, and then file a bug against the translation tool.

Yeah, I believe you're right. Noted as something else to file against the 
translation tool.

> src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n_de.properties
>  line 183:
> 
>> 181: jshell.fix.wrong.shortcut =Unerwartetes Zeichen nach 
>> Umschalt+Tab.\nVerwenden Sie "I" für automatischen Import, "V" zur 
>> Variablenerstellung oder "M" zur Methodenerstellung.\nWeitere Informationen 
>> finden Sie unter:\n/help shortcuts
>> 182: 
>> 183: help.usage = Verwendung:   jshell <option>... <load-file>...\nMögliche 
>> Optionen:\n    --class-path <path>   Gibt an, wo die Benutzerklassendateien 
>> gespeichert sind\n    --module-path <path>  Gibt an, wo die Anwendungsmodule 
>> gespeichert sind\n    --add-modules <module>(,<module>)*\n                   
>>        Gibt aufzulösende Module oder alle Module im\n                        
>>     Modulpfad an, wenn <module> ALL-MODULE-PATHs lautet\n    
>> --enable-native-access\n                          Ermöglicht Ausführung 
>> eingeschränkter nativer Methoden durch Code\n    --enable-preview      Code 
>> kann Vorschaufeatures in diesem Release nutzen\n    --startup <Datei>      
>> Ersetzung der Startdefinitionen mit einer Ausführung\n    --no-startup       
>>    Startdefinitionen werden nicht ausgeführt\n    --feedback <Modus>     
>> Gibt den anfänglichen Feedbackmodus an. Der Modus kann\n                     
>>        vordefiniert (Silent, Concise, Normal oder Verbose) oder\n            
>>                 
 vorab benutzerdefiniert sein\n    -q                    Stilles Feedback. 
Identisch mit: --feedback concise\n    -s                    Äußerst stilles 
Feedback. Identisch mit: --feedback silent\n    -v                    
Verbose-Feedback. Identisch mit: --feedback verbose\n    -J<flag>              
Übergibt <flag> an das Laufzeitsystem, hat aber keine Auswirkungen\n            
                auf die Ausführung von Code-Snippets. Um Kennzeichen 
anzugeben,\n                            die die Ausführung von Code-Snippets 
beeinflussen, verwenden Sie\n                            -R<flag>. Verwenden 
Sie alternativ dazu -J<flag> mit\n                            --execution 
local.\n    -R<flag>              Übergibt <flag> nur dann an das 
Laufzeitsystem, wenn\n                            Code-Snippets ausgeführt 
werden. Beispiel: -R-Dfoo=bar\n                            bedeutet, dass die 
Ausführung des Snippets\n                            System.getProperty("foo") 
"bar" zurückg
 ibt.\n    -C<flag>              Übergibt <flag> an den Java-Compiler in 
JShell.\n                            Beispiel: -C-Xlint aktiviert alle 
empfohlenen\n                           ...

This is what I've determined as well. I planned to leave these untranslated to 
reduce noise between drops as well.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19609#discussion_r1633762275
PR Review Comment: https://git.openjdk.org/jdk/pull/19609#discussion_r1632401147
PR Review Comment: https://git.openjdk.org/jdk/pull/19609#discussion_r1633748812
PR Review Comment: https://git.openjdk.org/jdk/pull/19609#discussion_r1632401256

Reply via email to