Fix Ventura Crash #3
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes WAR-3757
At a high level the issue that's causing this crash is that reading from the font traits
CFDictionary
in Core Text doesn't always return a value that matches theCFNumber
CFTypeID
(it returns aCFBoolean
that is alwaysfalse
instead).Confirmed in vanilla Objective-C the exact same behavior. As a simple fix, try to downcast into a
CFBoolean
if downcasting into aCFNumber
fails and then recreate aCFNumber
from thatCFBoolean
.Verified locally that Warp fully works on Ventura--I've used it as my day to day terminal for the past day with no issues.,
Note will also try to fix this upstream and merge the two versions of core-foundation-rs as a followup