Skip to content

Remove internal ICU support from databinding API#3985

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:icu
May 31, 2026
Merged

Remove internal ICU support from databinding API#3985
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:icu

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented May 7, 2026

The public ICU-based databinding API was removed in 2022 (commit 616243f). This change removes the remaining internal reflective fallbacks to com.ibm.icu.text.NumberFormat and com.ibm.icu.math.BigDecimal in org.eclipse.core.databinding, completing the transition to java.text. Java 17 (the bundle's BREE) handles BigDecimal parsing/formatting correctly via DecimalFormat#setParseBigDecimal(true), which is already used. Also drops two stale Javadoc references that contrasted these classes with sibling classes that no longer exist.

Planned for 4.41.

@eclipse-platform-bot
Copy link
Copy Markdown
Contributor

This pull request changes some projects for the first time in this development cycle.
Therefore the following files need a version increment:

bundles/org.eclipse.core.databinding/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch
From 317d53a26bd3c0051f3616a4e8e67fb68e1a5c71 Mon Sep 17 00:00:00 2001
From: Eclipse Platform Bot <platform-bot@eclipse.org>
Date: Thu, 7 May 2026 23:36:32 +0000
Subject: [PATCH] Version bump(s) for 4.40 stream


diff --git a/bundles/org.eclipse.core.databinding/META-INF/MANIFEST.MF b/bundles/org.eclipse.core.databinding/META-INF/MANIFEST.MF
index 12009435d8..da03b94e72 100644
--- a/bundles/org.eclipse.core.databinding/META-INF/MANIFEST.MF
+++ b/bundles/org.eclipse.core.databinding/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %pluginName
 Bundle-SymbolicName: org.eclipse.core.databinding
-Bundle-Version: 1.13.700.qualifier
+Bundle-Version: 1.13.800.qualifier
 Bundle-Vendor: %providerName
 Bundle-Localization: plugin
 Export-Package: org.eclipse.core.databinding;version="1.0.0",
-- 
2.53.0

Further information are available in Common Build Issues - Missing version increments.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Test Results

   864 files  ±0     864 suites  ±0   51m 48s ⏱️ -39s
 7 988 tests ±0   7 745 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 418 runs  ±0  19 763 ✅ ±0  655 💤 ±0  0 ❌ ±0 

Results for commit 1f2e2db. ± Comparison against base commit 53e3e92.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the icu branch 2 times, most recently from 6009db9 to 3efb1b0 Compare May 8, 2026 04:44
Public ICU-based databinding API was removed in 2022. This change
removes the remaining internal reflective support and documentation
references to com.ibm.icu, completing the transition to java.text.
@vogella vogella marked this pull request as ready for review May 30, 2026 08:24
@vogella vogella merged commit 1e6a7c9 into eclipse-platform:master May 31, 2026
18 checks passed
@vogella vogella deleted the icu branch May 31, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants