Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:4997 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 32370 invoked by uid 1010); 25 Oct 2003 19:43:21 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 32284 invoked from network); 25 Oct 2003 19:43:19 -0000 Received: from unknown (HELO mail.zend.com) (192.117.235.230) by pb1.pair.com with SMTP; 25 Oct 2003 19:43:19 -0000 Received: (qmail 10752 invoked from network); 25 Oct 2003 19:43:12 -0000 Received: from guardian.zend.office (HELO andi-laptop.zend.com) (10.1.1.4) by int.zend.com with SMTP; 25 Oct 2003 19:43:12 -0000 Message-ID: <5.1.0.14.2.20031025214003.034deab8@127.0.0.1> X-Sender: andi@127.0.0.1 X-Mailer: QUALCOMM Windows Eudora Version 5.1 Date: Sat, 25 Oct 2003 21:41:11 +0200 To: "Robert Richards" , In-Reply-To: <005101c39b26$d814cc10$f7dea8c0@cyberware.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; format=flowed Subject: Re: [PHP-DEV] simplexml and dom interop From: andi@zend.com (Andi Gutmans) References: <005101c39b26$d814cc10$f7dea8c0@cyberware.local> I have no problem with you committing this. It'd be good if at least another guy who has been working on the new XML extensions gives you thumbs up. Andi At 02:36 PM 10/25/2003 -0400, Robert Richards wrote: >If anyone is playing around with simplexml, dom and/or xsl, I have a patch >for the interoperability between simplexml and dom. I am leary about >committing it right now as it has limited testing, beta 2 is being >released next week and I am leaving on tuesday for a week or two, so wont >be able to fix any issues that might arise from it. > >patch - http://www.ctindustries.net/patches/interop.diff.txt >example script - http://www.ctindustries.net/patches/interop.php.txt > >The changes include: > simplexml > new function: simplexml_import_dom(domnode) returns an sxe object > behavior change: setting the value of an element will do just > that. I removes all child nodes unlike previous version which would > change the value of the first child node of the reference node (I dont > believe this was the intended behavior) > the change in behavior and the integration with the functions > from the libxml extension should resolve a lot of the libxml memory leaks > > dom > new function: dom_import_simplexml(sxe) returns domnode > also a minor change to clonenode method of domnode > >Let me know if people really want this in the beta release otherwise i am >going to wait until after I get back to further test and commit this stuff. > >Rob