Skip to content

ICU-23513 Fixed PDI overrides in bidi - #4205

Open
roozbehp wants to merge 1 commit into
unicode-org:mainfrom
roozbehp:roozbehp-bidi-j1
Open

ICU-23513 Fixed PDI overrides in bidi#4205
roozbehp wants to merge 1 commit into
unicode-org:mainfrom
roozbehp:roozbehp-bidi-j1

Conversation

@roozbehp

@roozbehp roozbehp commented Sep 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Apply the final X6a override step after processing the PDI's stack entry.
  • Retain the isolate identity for X10 and whitespace treatment for L1.
  • Add unit tests.

Note that this pull request only addresses the J1 case.

Checklist

  • Required: Issue filed: ICU-23513
  • Required: The PR title must be prefixed with a JIRA Issue number. Example: "ICU-NNNNN Fix xyz"
  • Required: Each commit message must be prefixed with a JIRA Issue number. Example: "ICU-NNNNN Fix xyz"
  • Issue accepted (done by Technical Committee after discussion)
  • Tests included, if applicable
  • API docs and/or User Guide docs changed or added, if applicable
  • Approver: Feel free to merge on my behalf

@roozbehp
ICU-23513 Fixed PDI overrides in bidi
9990400 
- Apply the final X6a override step after processing the PDI's stack entry.
- Retain the isolate identity for X10 and whitespace treatment for L1.
- Add unit tests.

Note that this patch only addresses the J1 case.
@roozbehp roozbehp mentioned this pull request Sep 18, 2026
Draft
7 tasks
@roozbehp

roozbehp commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

CC @eggrobin @markusicu

@markusicu markusicu added the after-release Do not merge until after an upcoming release or at least until its maintenance branch has been cut. label Sep 21, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eggrobin eggrobin Awaiting requested review from eggrobin
@markusicu markusicu Awaiting requested review from markusicu

At least 1 approving review is required to merge this pull request.

Assignees

@eggrobin eggrobin

Labels

after-release Do not merge until after an upcoming release or at least until its maintenance branch has been cut.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@roozbehp @eggrobin @markusicu