09:00
<Ashley Claymore>
I did report this Symbol.for stress test bug https://issues.chromium.org/issues/42202520
09:03
<Ashley Claymore>
issue is open but patch looks approved so not sure if fixed or not
09:04
<Ashley Claymore>
looks like the patch was applied
13:49
<kriskowal>
This suggests to me that V8 holds symbols strongly, before and after this patch. Is that right?
15:30
<Ashley Claymore>
that is my understanding yes
15:30
<Ashley Claymore>
for registered symbols
15:34
<Ashley Claymore>
Compared to SpiderMonkey's: https://searchfox.org/mozilla-central/source/js/src/vm/SymbolType.h#146 where the map key is weak
17:10
<kriskowal>
Thank you, that’s helpful.