<html>
<head>
<base href="https://bugs.webkit.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - WebAssembly: several tests added in r214504 crash when building with GCC"
href="https://bugs.webkit.org/show_bug.cgi?id=170390#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - WebAssembly: several tests added in r214504 crash when building with GCC"
href="https://bugs.webkit.org/show_bug.cgi?id=170390">bug 170390</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>(In reply to Saam Barati from <a href="show_bug.cgi?id=170390#c6">comment #6</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=306063&action=diff" name="attach_306063" title="Updated patch">attachment 306063</a> <a href="attachment.cgi?id=306063&action=edit" title="Updated patch">[details]</a></span>
> Updated patch
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=306063&action=review">https://bugs.webkit.org/attachment.cgi?id=306063&action=review</a>
>
> > Source/JavaScriptCore/wasm/js/WebAssemblyPrototype.cpp:94
> > + p.vm().promiseDeferredTimer->scheduleWorkSoon(promise, [source, promise, globalObject, plan = makeRefPtr(p)] () mutable {
>
> Why not plan=WTFMove(plan)?</span >
Because I've removed the line that created the plan to simplify it.
RefPtr<Plan> plan = makeRef(p);
[plan = WTFMove(plan)]
is equivalent to
[plan = makeRefPtr(p)]
I think this way is simpler, since we were only creating the plan to transfer it to the lambda capture (and to get the vm, but that is what caused the crash).</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>