Skip to content

ICU-23397 UBA: fail on ridiculously large numbers of runs.#3992

Merged
eggrobin merged 1 commit into
unicode-org:mainfrom
eggrobin:ICU-22397
May 20, 2026
Merged

ICU-23397 UBA: fail on ridiculously large numbers of runs.#3992
eggrobin merged 1 commit into
unicode-org:mainfrom
eggrobin:ICU-22397

Conversation

@eggrobin
Copy link
Copy Markdown
Member

@eggrobin eggrobin commented May 19, 2026 •
edited
Loading

Checklist

  • Required: Issue filed: ICU-23397
  • 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

@eggrobin eggrobin requested review from markusicu and roubert May 19, 2026 19:48
@eggrobin eggrobin changed the title ICU-22397 UBA: fail on ridiculously large numbers of runs. ICU-23397 UBA: fail on ridiculously large numbers of runs. May 19, 2026
@jira-pull-request-webhook
Copy link
Copy Markdown

jira-pull-request-webhook Bot commented May 19, 2026

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

roubert
roubert requested changes May 19, 2026
View reviewed changes
Comment thread icu4c/source/test/cintltst/cbiditst.c Outdated
roubert
roubert approved these changes May 20, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@roubert roubert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread icu4c/source/test/cintltst/cbiditst.c Outdated
@jira-pull-request-webhook
Copy link
Copy Markdown

jira-pull-request-webhook Bot commented May 20, 2026

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

@eggrobin eggrobin merged commit 6f0e464 into unicode-org:main May 20, 2026
98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@roubert roubert roubert approved these changes

@markusicu markusicu Awaiting requested review from markusicu

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@eggrobin @roubert