diff options
author | Arseny Kapoulkine <arseny.kapoulkine@gmail.com> | 2014-11-20 23:39:40 -0800 |
---|---|---|
committer | Arseny Kapoulkine <arseny.kapoulkine@gmail.com> | 2014-11-20 23:39:40 -0800 |
commit | 125aa55061ccde4ae7351a9a6c7270a15c9e0204 (patch) | |
tree | 266a4d0fdd7687630a299da1a8493cacbdbe3ae9 /tests/test_parse_doctype.cpp | |
parent | a0dc468170a9f429446feafea48fc73ae348b648 (diff) |
Fix node_declaration copying with empty name
node_copy_string relied on the fact that target node had an empty name and
value. Normally this is a safe assumption (and a good one to make since it
makes copying faster), however it was not checked and there was one case when
it did not hold.
Since we're reusing the logic for inserting nodes, newly inserted declaration
nodes had the name set automatically to xml, which in our case violates the
assumption and is counter-productive since we'll override the name right after
setting it.
For now the best solution is to do the same insertion manually - that results
in some code duplication that we can refactor later (same logic is partially
shared by _move variants anyway so on a level duplicating is not that bad).
Diffstat (limited to 'tests/test_parse_doctype.cpp')
0 files changed, 0 insertions, 0 deletions