Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ICU-22713 Delete layout/DataDrivenTest #3313

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

ICU-22713 Delete layout/DataDrivenTest #3313

wants to merge 3 commits into from

Conversation

roubert
Copy link
Member

@roubert roubert commented Dec 23, 2024

Checklist

  • Required: Issue filed: ICU-22713
  • Required: The PR title must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
  • Required: The PR description must include the link to the Jira Issue, for example by completing the URL in the first checklist item
  • Required: Each commit message must be prefixed with a JIRA Issue number. Example: "ICU-1234 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

ALLOW_MANY_COMMITS=true

@roubert
ICU-22713 Delete long since dead code.
920f4c6 
None of the code in this directory is ever compiled unless USING_ICULEHB
is set, so all such conditionally compiled code has been dead ever since
commit 4136a2f deleted the layout engine.
@roubert
ICU-22713 Delete unused files.
4285b27 
It's unclear when (if ever) these files were used for anything, but they
aren't used now and aren't expected to be used in the future.
@roubert
ICU-22713 Delete layout/DataDrivenTest.
fc5ff8d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@roubert