Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6b235623ef129c9a5533b1977160cba712d9cf38
      
https://github.com/WebKit/WebKit/commit/6b235623ef129c9a5533b1977160cba712d9cf38
  Author: Adrian Taylor <[email protected]>
  Date:   2025-11-13 (Thu, 13 Nov 2025)

  Changed paths:
    M Configurations/CommonBase.xcconfig
    M Source/WebKit/Configurations/Base.xcconfig
    M Source/WebKit/Configurations/BaseExtension.xcconfig
    M Source/WebKit/UIProcess/Cocoa/WKURLSchemeHandlerAdapter.swift
    M Source/WebKit/_WebKit_SwiftUI/Implementation/CocoaWebViewAdapter.swift
    M Source/WebKit/_WebKit_SwiftUI/Implementation/WebViewRepresentable.swift

  Log Message:
  -----------
  Enforce strict memory safety across WebKit
https://bugs.webkit.org/show_bug.cgi?id=300439
rdar://162275605

Reviewed by Elliott Williams.

Other WebKit configuration files promote some (few) classes of Swift warnings
to errors. These errors were being suppressed in some cases; this removes that
suppression.

The only class of warning being promoted to error is the strict memory safety
warnings; these become errors. This PR therefore also fixes the remaining
strict memory safety issues.

This is the third attempt at removing this suppression. The first attempt
caused trouble due to these safety errors which did not show up on the EWS bots
in use at the time. EWS is now building using SDK versions which should
correctly show up any remaining problems. The second attempt failed because
VisionOS has some remaining code that does not comply with strict memory
safety; the bug numbers for fixing that are mentioned in a comment.

It is worth noting that PAL.xcconfig still overrides the
OTHER_SWIFT_FLAGS variable and thus does not get the benefit of thesse safety
errors. That's intentional, because PAL still has some strict safety problems
to be resolved, tracked in rdar://164560176.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to