RESOLVED FIXED 233539
[JSC] Add Intl.NumberFormat.formatRangeToParts 
https://bugs.webkit.org/show_bug.cgi?id=233539
Summary [JSC] Add Intl.NumberFormat.formatRangeToParts
Yusuke Suzuki
Reported 2021-11-27 23:11:00 PST
[JSC] Add Intl.NumberFormat.formatRangeToParts
Attachments
Patch (21.67 KB, patch)
2021-11-27 23:11 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (47.03 KB, patch)
2021-11-28 21:44 PST, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
Patch (47.92 KB, patch)
2021-11-28 22:12 PST, Yusuke Suzuki 		ross.kirsling: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2021-11-27 23:11:14 PST
Created attachment 445234 [details] Patch
Yusuke Suzuki
Comment 2 2021-11-28 21:44:17 PST
Created attachment 445248 [details] Patch
Yusuke Suzuki
Comment 3 2021-11-28 22:12:35 PST
Created attachment 445249 [details] Patch
Ross Kirsling
Comment 4 2021-11-29 12:11:10 PST
Comment on attachment 445249 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445249&action=review r=me, very nice. > Source/JavaScriptCore/runtime/IntlNumberFormat.cpp:1511 > + auto partValue = jsSubstring(vm, formatted, field.m_range.begin(), field.m_range.end() - field.m_range.begin()); Wonder if we should add a method to WTF::Range for this?
Yusuke Suzuki
Comment 5 2021-11-29 12:15:50 PST
Comment on attachment 445249 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445249&action=review >> Source/JavaScriptCore/runtime/IntlNumberFormat.cpp:1511 >> + auto partValue = jsSubstring(vm, formatted, field.m_range.begin(), field.m_range.end() - field.m_range.begin()); > > Wonder if we should add a method to WTF::Range for this? Added Range::distance method, which is aligned to STL's naming (begin, end, distance).
Yusuke Suzuki
Comment 6 2021-11-29 12:29:13 PST
Committed r286255 (244618@main): <https://commits.webkit.org/244618@main>
Radar WebKit Bug Importer
Comment 7 2021-11-29 12:30:23 PST
<rdar://problem/85829885>
Note You need to log in before you can comment on or make changes to this bug.