aniani/elinks
1
0
Fork 0
mirror of https://github.com/rkd77/elinks.git synced 2024-09-19 01:36:33 -04:00
Code

[nodelist2] Free nodelist after generating values.

Browse Source
This commit is contained in:
Witold Filipczyk 2024-07-23 19:01:50 +02:00
parent 68a7a26418
commit c52510fd9f
4 changed files with 16 additions and 43 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/ecmascript/quickjs/document.c
View File

@ -1692,7 +1692,11 @@ js_document_querySelectorAll(JSContext *ctx, JSValueConst this_val, int argc, JS
JS_FreeCString(ctx, selector);
//dom_node_unref(doc);
return getNodeList2(ctx, result_list);
JSValue rr = getNodeList2(ctx, result_list);
free_list(*result_list);
mem_free(result_list);
return rr;
}
#if 0

6
src/ecmascript/quickjs/element.c
View File

@ -3289,7 +3289,11 @@ js_element_querySelectorAll(JSContext *ctx, JSValueConst this_val, int argc, JSV
JS_FreeCString(ctx, selector);
//dom_node_unref(el);
return getNodeList2(ctx, result_list);
JSValue rr = getNodeList2(ctx, result_list);
free_list(*result_list);
mem_free(result_list);
return rr;
}
static JSValue

44
src/ecmascript/quickjs/nodelist2.c
View File

@ -56,26 +56,6 @@ static JSClassID js_nodelist2_class_id;
void *map_nodelist2;
void *map_rev_nodelist2;
static void *
js_nodeList2_GetOpaque(JSValueConst this_val)
{
REF_JS(this_val);
return attr_find_in_map_rev(map_rev_nodelist2, this_val);
}
static void
js_nodeList2_SetOpaque(JSValueConst this_val, void *node)
{
REF_JS(this_val);
if (!node) {
attr_erase_from_map_rev(map_rev_nodelist2, this_val);
} else {
attr_save_in_map_rev(map_rev_nodelist2, this_val, node);
}
}
#if 0
static JSValue
js_nodeList2_get_property_length(JSContext *ctx, JSValueConst this_val)
@ -102,25 +82,10 @@ js_nodeList2_item(JSContext *ctx, JSValueConst this_val, int argc, JSValueConst
if (argc != 1) {
return JS_UNDEFINED;
}
int index;
JS_ToInt32(ctx, &index, argv[0]);
LIST_OF(struct selector_node) *sni = (LIST_OF(struct selector_node) *)(js_nodeList2_GetOpaque(this_val));
int counter = 0;
struct selector_node *sn = NULL;
uint32_t index;
JS_ToUint32(ctx, &index, argv[0]);
foreach (sn, *sni) {
if (counter == index) {
break;
}
counter++;
}
if (!sn || !sn->node) {
return JS_NULL;
}
JSValue ret = getElement(ctx, sn->node);
return ret;
return JS_GetPropertyUint32(ctx, this_val, index);
}
static JSValue
@ -161,9 +126,6 @@ getNodeList2(JSContext *ctx, void *nodes)
JS_SetPropertyFunctionList(ctx, proto, js_nodeList2_proto_funcs, countof(js_nodeList2_proto_funcs));
JS_SetClassProto(ctx, js_nodelist2_class_id, proto);
attr_save_in_map(map_nodelist2, nodes, proto);
js_nodeList2_SetOpaque(proto, nodes);
LIST_OF(struct selector_node) *sni = (LIST_OF(struct selector_node) *)nodes;
struct selector_node *sn = NULL;

3
test/ecmascript/assert/element.querySelectorAll.html
View File

@ -16,6 +16,9 @@ function myFunction() {
var y = el.querySelectorAll('.b');
console.assert(y.length === 1, 'b class b.');
console.assert(y.item(0).innerText === 'This b class.', 'b class');
}
console.error('element.querySelectorAll.html');