Patch to make Javascript writeln() work (attempt 3)

This patch to Document/bnd makes the Javascript writeln() function work as well as write(). This version inserts "\n", which seems to be what the language requires. <br> was wrong. It duplicates code from write(), but I'm really not keen on Daniel's alternative suggestion. Sorry. (Re. the earlier thread: I think it pays a third price, that of being less clear than duplicating code.) Hope this helps. Dave diff --git a/javascript/duktape/Document.bnd b/javascript/duktape/Document.bnd index 6d11ea9..a97f30e 100644 --- a/javascript/duktape/Document.bnd +++ b/javascript/duktape/Document.bnd @@ -13,6 +13,7 @@ class Document { #include "utils/corestrings.h" #include "render/html_internal.h" #include "utils/libdom.h" +#include "utils/utils.h" %}; } @@ -37,6 +38,27 @@ method Document::write() return 0; %} +method Document::writeln() +%{ + const char nl[] = "\n"; + struct html_content *htmlc; + duk_size_t text_len; + for (int i = 0; i < duk_get_top(ctx); ++i) + duk_safe_to_string(ctx, i); + duk_concat(ctx, duk_get_top(ctx)); + const char *text = duk_safe_to_lstring(ctx, 0, &text_len); + LOG("Writeln %*s", (int)text_len, text); + dom_exception err; + err = dom_node_get_user_data(priv->parent.node, + corestring_dom___ns_key_html_content_data, + &htmlc); + if (err == DOM_NO_ERR && htmlc->parser != NULL) { + dom_hubbub_parser_insert_chunk(htmlc->parser, (uint8_t *)text, text_len); + dom_hubbub_parser_insert_chunk(htmlc->parser, (uint8_t *)nl, SLEN(nl)); + } + return 0; +%} + method Document::createTextNode() %{ dom_node *newnode; ____________________________________________________________ FREE 3D EARTH SCREENSAVER - Watch the Earth right on your desktop! Check it out at http://www.inbox.com/earth