Stop using UncheckedLock in JSC::VMInspector, as it is being phased out in favor of Lock, which supports Clang thread safety analysis.
Created attachment 430115 [details] Patch
Created attachment 430117 [details] Patch
Comment on attachment 430117 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=430117&action=review r=me > Source/WTF/wtf/Lock.cpp:35 > +#include <unistd.h> Is the #include in VMInspector.cpp still needed? If not, can you remove it?
Created attachment 430129 [details] Patch
(In reply to Mark Lam from comment #3) > Comment on attachment 430117 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=430117&action=review > > r=me > > > Source/WTF/wtf/Lock.cpp:35 > > +#include <unistd.h> > > Is the #include in VMInspector.cpp still needed? If not, can you remove it? Done.
Committed r278251 (238288@main): <https://commits.webkit.org/238288@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 430129 [details].
<rdar://problem/78657229>