[Webkit-unassigned] [Bug 151917] Implement UCharIterator StringView::createIterator()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=151917

--- Comment #4 from Darin Adler <darin at apple.com> ---
(In reply to comment #3)
> Could you please advise me on how to go about this? Should I just change it
> to a function that takes a StringView while keeping it in StringView.h, or
> should I not do this in general? If the latter, is there a way to avoid
> duplicate code?

As a first step, I’d suggest changing it to a function that takes a StringView and putting the function in Collator.h. Later we can move it somewhere else if we want to use UCharIterator for things other than collation.

The Collator abstraction is not super-important since we are OK with WebKit using ICU directly. It was created by someone who was trying to change WebKit to work with multiple international libraries, but that’s not really a goal for the project now.

Anyway, I think having UCharIterator createUCharIterator(StringView) in Collator.h would be a fine way to share it with other code.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20151214/3275fea8/attachment.html>