[webkit-changes] [WebKit/WebKit] 115f23: JSStringCreateWithCFString can now use CFIndex

Rose noreply at github.com
Thu Feb 9 13:50:45 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 115f2370e6f35139cb1672d1b4cc89fd0d3d0121
      https://github.com/WebKit/WebKit/commit/115f2370e6f35139cb1672d1b4cc89fd0d3d0121
  Author: Rose <83477269+AtariDreams at users.noreply.github.com>
  Date:   2023-02-09 (Thu, 09 Feb 2023)

  Changed paths:
    M Source/JavaScriptCore/API/JSStringRefCF.cpp

  Log Message:
  -----------
  JSStringCreateWithCFString can now use CFIndex
https://bugs.webkit.org/show_bug.cgi?id=251619

<rdar://problem/6806478>
Reviewed by Yusuke Suzuki.

The original size_t 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 about it.

*Source/JavaScriptCore/API/JSStringRefCF.cpp: JSStringRef JSStringCreateWithCFString(CFStringRef string):

Canonical link: https://commits.webkit.org/260083@main




More information about the webkit-changes mailing list