Functions in a namespace have the wrong id

[rhdunn]

Given:

namespace foo { void bar(); }

The function has the entry:

<function id="bar" name="bar">...</function>

This is wrong, as the id is not fully qualified. It should be:

<function id="foo::bar" name="bar">...</function>

Removing the qid property from cldoc/nodes/function.py fixes this issue, but I don't know why this override was created in the first place (note: method.py overrides qid to use the Node version and not the incorrect Function version).

[jessevdk]

This has been fixed in master (including tests). Actually, the proposed fix in pull request #49 is incorrect. The reason that the qid of the function is overridden is because cldoc will move free functions that look like they belong to a struct (C API) as a child of the struct. The id however should not include the struct as a parent. This is of course wrong then in the case of a namespace, which is now fixed.