swift
[cxx-interop] do not rely on bodyParams being always non-null when im…
#76286
Merged

[cxx-interop] do not rely on bodyParams being always non-null when im… #76286

hyp merged 1 commit into swiftlang:main from hyp:eng/bodyParamsimport
hyp
hyp309 days ago

…porting function's name

This change fixes a swift-ide-test crash that occured in the Interop\Cxx\stdlib\msvcprt-module-interface.swift testcase with a newer MSVC, as one of its operator() had a parameter with a type that couldn't have been imported. The change ensures that body params are not used if they're null.

hyp [cxx-interop] do not rely on bodyParams being always non-null when im…
86886d96
hyp hyp requested a review from compnerd compnerd 309 days ago
hyp hyp requested a review from hjyamauchi hjyamauchi 309 days ago
hyp hyp requested a review from zoecarver zoecarver 309 days ago
hyp hyp requested a review from egorzhdan egorzhdan 309 days ago
hyp hyp requested a review from beccadax beccadax 309 days ago
hyp hyp requested a review from ian-twilightcoder ian-twilightcoder 309 days ago
hyp
hyp309 days ago

@swift-ci please test

compnerd
compnerd approved these changes on 2024-09-05
compnerd
compnerd309 days ago

@swift-ci please test Windows platform

hjyamauchi
hjyamauchi approved these changes on 2024-09-05
egorzhdan
egorzhdan approved these changes on 2024-09-05
egorzhdan308 days ago

LGTM!

hyp hyp merged 8ea7569b into main 305 days ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone