TIBCOSoftware / genxdm

GenXDM: XQuery/XPath Data Model API, bridges, and processors for tree-model neutral access to XML.
http://www.genxdm.org/
9 stars 4 forks source link

Remove copyNamespaces parameter from Model.stream() #93

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
It is not clear what purpose is served by ever passing false to copyNamespaces. 
It strips namespaces. Why would you do that?

Discussed on the mailing list (more may get pasted in a comment; I'm 
summarizing, here).

Possible solutions:

Create a new interface, so that we can keep backward compatibility. I'm opposed 
very strongly; I think that's something that has to be done post-1.0 release, 
not before it.

Deprecate the current method and make its copyNamespaces=false behavior go 
away. Add a method without that parameter, and move the current implementations 
there (without copyNamespaces handling) in all implementations.

Just drop copyNamespaces, with no deprecation warning.

I'd generally prefer the second option, but don't mind the third too much.

Original issue reported on code.google.com by aale...@gmail.com on 2 Aug 2012 at 6:16

GoogleCodeExporter commented 9 years ago
Started. The typed versions have been stripped of their overrides.

Original comment by aale...@gmail.com on 2 Aug 2012 at 8:02

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r351.

Original comment by aale...@gmail.com on 6 Aug 2012 at 8:25