Re: [PATCH v3 0/9] MR3300: jscript: WeakMap implementation.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/jscript/set.c:
>      jsval_t *r)
>
> {
> - FIXME("\n");
> - return E_NOTIMPL;
> - struct weakmap_entry *entry, *entry2;
> - WeakMapInstance *weakmap;
> - HRESULT hres;
> -
> - hres = get_weakmap_this(ctx, vthis, &weakmap);
> - if(FAILED(hres))
> -
>    return hres;
> -
> - TRACE("%p\n", weakmap);
> -
> - RB_FOR_EACH_ENTRY_DESTRUCTOR(entry, entry2, &weakmap->map, struct weakmap_entry, entry)
> -
>    release_weakmap_entry(entry);

Same as others, this is not safe.