8734 2006-05-04 11:02:59 -0700 Would like a Vector::append that takes another Vector 2006-05-04 13:32:52 -0700 1 1 1 Unclassified WebKit Web Template Framework 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 6638 1 andersca andersca oldest_to_newest 41250 0 andersca 2006-05-04 11:02:59 -0700 The xpath implementation uses a lot of Vectors for keeping track of node sets. It'd be nice to have an append function that takes another Vector as this it's something that's done commonly in the xpath code 41251 1 8107 andersca 2006-05-04 11:07:03 -0700 Created attachment 8107 Implement function This function works for me, but I'm not sure how correct it is. It doesn't support appending the vector to itself, but I don't know if we need that 41264 2 8107 mjs 2006-05-04 13:17:19 -0700 Comment on attachment 8107 Implement function Looks good to me. A more general version that takes start and end iterator might also be handy (and could be used to implement this one. 8107 2006-05-04 11:07:03 -0700 2006-05-04 13:17:19 -0700 Implement function vector-append.txt text/plain 1505 andersca SW5kZXg6IGt4bWxjb3JlL1ZlY3Rvci5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIGt4bWxjb3JlL1ZlY3Rvci5o CShyZXZpc2lvbiAxNDE3OSkKKysrIGt4bWxjb3JlL1ZlY3Rvci5oCSh3b3JraW5nIGNvcHkpCkBA IC0zNzgsNiArMzc4LDcgQEAgbmFtZXNwYWNlIEtYTUxDb3JlIHsKICAgICAgICAgdm9pZCBjbGVh cigpIHsgcmVzaXplKDApOyB9CiAKICAgICAgICAgdGVtcGxhdGU8dHlwZW5hbWUgVT4gdm9pZCBh cHBlbmQoY29uc3QgVSYpOworICAgICAgICB0ZW1wbGF0ZTx0eXBlbmFtZSBVPiB2b2lkIGFwcGVu ZChjb25zdCBWZWN0b3I8VT4mKTsKICAgICAgICAgdGVtcGxhdGU8dHlwZW5hbWUgVT4gdm9pZCBp bnNlcnQoc2l6ZV90IHBvc2l0aW9uLCBjb25zdCBVJik7CiAgICAgICAgIHZvaWQgcmVtb3ZlKHNp emVfdCBwb3NpdGlvbik7CiAKQEAgLTU0NSw2ICs1NDYsMTUgQEAgbmFtZXNwYWNlIEtYTUxDb3Jl IHsKICAgICB9CiAKICAgICB0ZW1wbGF0ZTx0eXBlbmFtZSBULCBzaXplX3QgaW5saW5lQ2FwYWNp dHk+IHRlbXBsYXRlPHR5cGVuYW1lIFU+CisgICAgaW5saW5lIHZvaWQgVmVjdG9yPFQsIGlubGlu ZUNhcGFjaXR5Pjo6YXBwZW5kKGNvbnN0IFZlY3RvcjxVPiYgdmFsKQorICAgIHsKKyAgICAgICAg aWYgKHNpemUoKSArIHZhbC5zaXplKCkgPj0gY2FwYWNpdHkoKSkKKyAgICAgICAgICAgIGV4cGFu ZENhcGFjaXR5KHNpemUoKSArIHZhbC5zaXplKCkpOworICAgICAgICBmb3IgKHVuc2lnbmVkIGkg PSAwOyBpIDwgdmFsLnNpemUoKTsgaSsrKQorICAgICAgICAgICAgYXBwZW5kKHZhbFtpXSk7Cisg ICAgfQorICAgIAorICAgIHRlbXBsYXRlPHR5cGVuYW1lIFQsIHNpemVfdCBpbmxpbmVDYXBhY2l0 eT4gdGVtcGxhdGU8dHlwZW5hbWUgVT4KICAgICBpbmxpbmUgdm9pZCBWZWN0b3I8VCwgaW5saW5l Q2FwYWNpdHk+OjppbnNlcnQoc2l6ZV90IHBvc2l0aW9uLCBjb25zdCBVJiB2YWwpCiAgICAgewog ICAgICAgICBBU1NFUlQocG9zaXRpb24gPD0gc2l6ZSgpKTsKSW5kZXg6IENoYW5nZUxvZwo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBDaGFuZ2VMb2cJKHJldmlzaW9uIDE0MTc5KQorKysgQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTEgQEAKKzIwMDYtMDUtMDQgIEFuZGVycyBDYXJsc3NvbiAg PGFuZGVyc2NhQG1hYy5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku CisKKyAgICAgICAgKiBreG1sY29yZS9WZWN0b3IuaDoKKyAgICAgICAgKEtYTUxDb3JlOjo6OmFw cGVuZCk6CisgICAgICAgIE5ldyBmdW5jdGlvbiB0aGF0IHRha2VzIGFub3RoZXIgYXJyYXkuCisK IDIwMDYtMDUtMDIgIFN0ZXZlIEZhbGtlbmJ1cmcgIDxzZmFsa2VuQGFwcGxlLmNvbT4KIAogICAg ICAgICBSZXZpZXdlZCBieSBlcmljLgo=