RFR: 6928542: Chinese characters in RTF are not decoded [v7]

[Ichiroh Takiguchi]

On Thu, 26 Oct 2023 12:44:28 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Ichiroh Takiguchi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   6928542: Chinese characters in RTF are not decoded
>
> For me the added regression test still fails with the fix in WIndows 10...anything I need to do more as a prerequisite?
> 
> 
> Read data^M
> =========^M
> Gr\\u00fcezi -  Switzerland 0^M
> \\u0082\\u00b1\\u0082\\u00f1\\u0082\\u00c9\\u0082\\u00bf\\u0082\\u00cd - Japanese 128^M
> \\u00be\\u00c8\\u00b3\\u00e7\\u00c7\\u00cf\\u00bc\\u00bc\\u00bf\\u00e4 - Korean 129^M
> \\u00c4\\u00e3\\u00ba\\u00c3 - China 134^M
> \\u00bbO\\u00c6W - Traditional Chinese - Taiwan 136^M
> \\u00e3\\u00e5\\u00e9\\u00e1 \\u00f3\\u00ef\\u00f5 - Greek 161^M
> A\\u00f0a\\u00e7 - Turkish (Tree) 162^M
> \\u00fe - Vietnam currency 163^M
> \\u00f9\\u00c8\\u00d1\\u00ec\\u00e5\\u00c9\\u00ed - Hebrew 177^M
> \\u00e3\\u00d1\\u00cd\\u00c8\\u00c7 - Arabic 178^M
> A\\u00e8i\\u00fb - Lithuanian (Thank you) 186^M
> \\u00c7\\u00e4\\u00f0\\u00e0\\u00e2\\u00f1\\u00f2\\u00e2\\u00f3\\u00e9\\u00f2\\u00e5 - Russian 204^M
> \\u00ca\\u00c7\\u00d1\\u00ca\\u00b4\\u00d5 - Thailand 222^M
> cze\\uc48f - Polish 238^M
> ^M
> Expected data^M
> =============^M
> Gr\\u00fcezi -  Switzerland 0^M
> \\u3053\\u3093\\u306b\\u3061\\u306f - Japanese 128^M
> \\uc548\\ub155\\ud558\\uc138\\uc694 - Korean 129^M
> \\u4f60\\u597d - China 134^M
> \\u81fa\\u7063 - Traditional Chinese - Taiwan 136^M
> \\u03b3\\u03b5\\u03b9\\u03b1 \\u03c3\\u03bf\\u03c5 - Greek 161^M
> A\\u011fa\\u00e7 - Turkish (Tree) 162^M
> \\u20ab - Vietnam currency 163^M
> \\u05e9\\u05b8\\u05c1\\u05dc\\u05d5\\u05b9\\u05dd - Hebrew 177^M
> \\u0645\\u0631\\u062d\\u0628\\u0627 - Arabic 178^M
> A\\u010di\\u016b - Lithuanian (Thank you) 186^M
> \\u0417\\u0434\\u0440\\u0430\\u0432\\u0441\\u0442\\u0432\\u0443\\u0439\\u0442\\u0435 - Russian 204^M
> \\u0e2a\\u0e27\\u0e31\\u0e2a\\u0e14\\u0e35 - Thailand 222^M
> cze\\u015b\\u0107 - Polish 238^M
> ^M
> java.lang.RuntimeException: Test failed^M
>         at RTFReadFontCharsetTest.main(RTFReadFontCharsetTest.java:114)^

Hello @prsadhuk .
Sorry for bad reaction.
I tried to run my testcase on Japanese Windows 10/11.
It worked fine as expected.
Could you show me your windows environment ?

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

PR Comment: https://git.openjdk.org/jdk/pull/13553#issuecomment-1798831955