RFR: 8361644 : Update ICU4C to 77.1
Kevin Rushforth
kcr at openjdk.org
Tue Sep 9 20:54:14 UTC 2025
On Wed, 3 Sep 2025 11:21:24 GMT, Hima Bindu Meda <hmeda at openjdk.org> wrote:
> Updated icu to v77.1. Verified build on windows, mac and linux. Sanity testing looks fine. No issues seen.
Looks good with one suggested change.
modules/javafx.web/src/main/native/Source/ThirdParty/icu/source/common/unicode/platform.h line 740:
> 738: # else
> 739: // conformant C11
> 740: # define U_HAVE_CHAR16_T 0
Suggestion:
/* Commented out for JavaFX
// conformant C11
# define U_HAVE_CHAR16_T 1
*/
// JavaFX: We cannot define U_HAVE_CHAR16_T
# define U_HAVE_CHAR16_T 0
Since this alters upstream code, I recommend restoring line 740 from upstream (which sets `U_HAVE_CHAR16_T ` to 1), commenting it out, and adding a JavaFX-specific comment like the one I suggested above.
-------------
PR Review: https://git.openjdk.org/jfx/pull/1882#pullrequestreview-3203239751
PR Review Comment: https://git.openjdk.org/jfx/pull/1882#discussion_r2334737339
More information about the openjfx-dev
mailing list