Re: Should Document.cloneNode() work in Level 1?

"Don Park" <donpark@quake.net> writes:

> I would hate to see cloneNode disabled for Level 1.

Same here. 

> Perhaps there is something to be said for changing the cloneNode interface
> to:
> 
> Node cloneNode(boolean deep, Document targetDoc);

A better alternative would be to put this on Document rather than Node:

  Node cloneNode(Node original, boolean deep)

This would have a couple of advantages:

  1. it would keep _all_ the node-creation methods on Document
  2. it would not have to be modified or extended when new node types are
     added.  
  3. there are no problems when the original node's document and the target
     document use totally different implementation classes.

-- 
 Stephen R. Savitzky   Chief Software Scientist, Ricoh Silicon Valley, Inc., 
<steve@rsv.ricoh.com>                            California Research Center
 voice: 650.496.5710   fax: 650.854.8740    URL: http://rsv.ricoh.com/~steve/
  home: <steve@starport.com> URL: http://www.starport.com/people/steve/

Received on Tuesday, 8 September 1998 14:27:44 UTC