fix(macOS): panic when loading multiple dylibs (fix #1675)

[russellmcc]

closes #1675

This fixes the panic by using the new "auto-generating" name feature of define_class! objc2 - the name of the objective C class for the delegates will include the wry version number so will not be unsound to have multiple in the process.

I also added a unique name to the custom protocol handler, as this was causing issues even after the fix above.

I tested this with a local build of a few audio plug-ins using wry loaded into the same process

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	objc2@​0.6.0 ⏵ 0.6.4	-1

View full report

[pewsheen]

Thank you! Do you have a minimum reproducible demo or repo so we can have a quick test for it.

[russellmcc]

Thank you! Do you have a minimum reproducible demo or repo so we can have a quick test for it.

I can put something together!

[russellmcc]

Here's a repro of both issues:

• The crash
• Protocol incorrectly sharing between unrelated dylibs

https://github.com/russellmcc/wry-dylib-repro

[pewsheen]

Looks good! Your repro is clean and very helpful!

[github-actions]

Package Changes Through f3f9b27

There are 1 changes which include wry with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package	current	next
wry	0.54.2	0.54.3

Add another change file through the GitHub UI by following this link.

---

Read about change files or the docs at github.com/jbolda/covector