(In reply to Lina Cambridge (she/her) [:lina] from comment #0)
- Add a
nul_terminate!("foo") macro that asserts a static string doesn't contain embedded NULs, then adds a trailing one.
ThreadPtrHolder::new will likewise want to assert that the string is NUL-terminated.
I think this is probably the best option. Surely somebody has already written
nul_terminate! or something very like it.
ProxyReleaseEvent to hold an
nsCString instead of a
const char*. We'd need to change
This change would not be desirable, because then every runnable would need to construct and tear down an
nsCString, plus potential additional memory allocation. If the
nsCString requirement was contained to
ProxyReleaseEvent, that would be more doable.
ThreadPtrHolder::name entirely, and pass
ptr::null(). Proxied releases from Rust won't log runnable names, so this is (probably?) not ideal.
I'd prefer to avoid this.