On Wed, 9 Apr 2025 21:26:15 GMT, Justin Lu <j...@openjdk.org> wrote:

>> src/java.xml/share/classes/com/sun/org/apache/xml/internal/serializer/Encodings.properties
>>  line 22:
>> 
>>> 20: # Peter Smolik
>>> 21: Cp1250 WINDOWS-1250 0x00FF
>>> 22: # Patch attributed to hava...@underdusken.no (H�vard Wigtil)
>> 
>> This does not seem to have been a correct conversion.
>
> Right, that `å` looks to have been incorrectly converted during the 
> ISO-8859-1 to UTF-8 conversion. (I can't find the script used for conversion 
> as this change is from some time ago.)
> 
> Since the change occurs in a comment (thankfully), it should be harmless and 
> the next upstream update of this file would overwrite this incorrect change. 
> However, this file does not seem to be updated that often, so I can also file 
> an issue to correct this if you would prefer that.

You don't have to do that, I'm working on an omnibus UTF-8 fixing PR right now, 
where I will include a fix for this as well.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/12726#discussion_r2036695622

Reply via email to