WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
251619
JSStringCreateWithCFString can now use CFIndex
https://bugs.webkit.org/show_bug.cgi?id=251619
Summary
JSStringCreateWithCFString can now use CFIndex
Seija K.
Reported
2023-02-02 11:00:19 PST
This workaround came about 14 years ago, right when Apple was STILL compiling for PowerPC. Basically, so many devices ran on 32 bits. A long type can hold onto the required length now, so we no longer need to worry.
Attachments
Add attachment
proposed patch, testcase, etc.
Seija K.
Comment 1
2023-02-06 11:26:20 PST
Pull request:
https://github.com/WebKit/WebKit/pull/9542
Radar WebKit Bug Importer
Comment 2
2023-02-09 11:01:24 PST
<
rdar://problem/105230631
>
EWS
Comment 3
2023-02-09 13:50:36 PST
Committed
260083@main
(115f2370e6f3): <
https://commits.webkit.org/260083@main
> Reviewed commits have been landed. Closing PR #9542 and removing active labels.
Yusuke Suzuki
Comment 4
2023-02-10 12:46:13 PST
Reverted by
https://github.com/WebKit/WebKit/pull/9941
Yusuke Suzuki
Comment 5
2023-02-10 12:46:16 PST
Re-opening for pull request
https://github.com/WebKit/WebKit/pull/9941
EWS
Comment 6
2023-02-10 12:56:17 PST
Committed
260125@main
(27d620c4a193): <
https://commits.webkit.org/260125@main
> Reviewed commits have been landed. Closing PR #9941 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug