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
267210
Remove unused 'CStringTranslator' in StringImpl.h
https://bugs.webkit.org/show_bug.cgi?id=267210
Summary
Remove unused 'CStringTranslator' in StringImpl.h
Ahmad Saleem
Reported
2024-01-07 11:44:10 PST
Hi Team, While looking into 'StringImpl.h', it seems we have following two references in code:
https://searchfox.org/wubkat/rev/88bdb08baf23b66d0495fe895e251cc5fb8c40e6/Source/WTF/wtf/text/StringImpl.h#189
and
https://searchfox.org/wubkat/rev/88bdb08baf23b66d0495fe895e251cc5fb8c40e6/Source/WTF/wtf/text/StringImpl.h#69
Looking into it, we don't have any other references to it. So I think we can safely remove these unless if there are any internal use cases etc. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
EWS
Comment 1
2024-01-08 01:54:03 PST
Committed
272755@main
(3db3a794a844): <
https://commits.webkit.org/272755@main
> Reviewed commits have been landed. Closing PR #22496 and removing active labels.
Radar WebKit Bug Importer
Comment 2
2024-01-08 01:55:14 PST
<
rdar://problem/120634868
>
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