icu icon indicating copy to clipboard operation
icu copied to clipboard
verified publisher icon unicode-org

Require Range to be a range in UTFStringCodePoints to improve error messages

Open eggrobin opened this issue 2 months ago • 2 comments

Suggested by @jmr after seeing the error messages generated by UTFStringCodePoints<..., char> (which was how this API worked in an earlier draft).

TODO: Fill out the checklist below.

Checklist

  • [ ] Required: Issue filed: ICU-NNNNN
  • [ ] Required: The PR title must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
  • [ ] 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

eggrobin avatar Oct 22 '25 14:10 eggrobin

@markusicu, should this target main on a new ticket, or 78 maintenance on a reopened ICU-23004?

eggrobin avatar Oct 22 '25 14:10 eggrobin

@markusicu, should this target main on a new ticket, or 78 maintenance on a reopened ICU-23004?

@eggrobin is this PR fixing problems, or just "improv[ing] error messages"? If it's an important bug fix, then it should go onto the maintenance branch, with the existing ticket. If it's merely an improvement, then it should go onto main for ICU 79.

@mihnita @richgillam FYI

markusicu avatar Oct 23 '25 15:10 markusicu