The C implementation of ElementTree which is the default by python3.3 [1] does not accept the namespaces keyword for Element/ElementTree.find(all|iter|...) and maybe others.


[1] http://bugs.python.org/issue13988

As I wrote on the other issue:

> It seems to me that namespaces are actually supported, but they are
> accepted only as positional args and not keyword args, so this should 
> be easy to fix.

I can prepare a patch unless someone else is faster than me.

Thanks for the report - such regressions are taken seriously and will be fixed.

Ezio - you can go ahead and prepare a patch. I'm currently away from home (business trip) but I will look into it when I get back next weak.

The attached patch adds support for keyword args to find, findtext, findall, and iterfind.  iter and itertext don't seem to accept a namespace argument.  The patch includes minimal tests.

The name I used for the kwargs are the same of the Python version.  The documentation uses different names, and extra args (like "namespaces") are not even documented.

I didn't have time yet to double-check if other functions needs to be updated and to add better tests that actually check for specific namespaces.

Applied the patch, but could not verify 'it works for me' as Element lacks the attrib keyword.

Attached patch fixes Element constructor to accept 'attrib' as keyword arg.  I couldn't find a way to make this work using PyArg_ParseTupleAndKeywords, so I ended up parsing the kwds by hand.
While adding more tests I found out another difference with the Python version.  The C version raises a TypeError if attrib is not a dict, whereas the Python version raises an AttributeError while attempting to do a .copy() of the object.  I changed the Python version to raise a TypeError too.

SubElement needs to handle the attrib keyword too.

Ezio,

Your patch looks good, with these notes:

* When raising TypeError in the Python constructor, say that 'attrib' must be a dict, otherwise it's not clear *what* must be a dict.
* The 'attrib' keyword handling has to be added to SubElement too, as Markus said. I guess it would make sense to move it into some helper function to avoid duplication.
* The new keyword arguments have to be documented in the ReST docs of ET, since they haven't been documented until now

The whole namespace issue is not very well documented in the ReST doc for ET - should open a new issue on this

Updated patch with fixed error messages, additional tests and some documentation. No support in SubElement yet

Committed all fixes in http://hg.python.org/cpython/rev/7d252dbfbee3