Doxygen is missing some methods because it's not smart enough about namespaces and sometimes thinks that the signatures of the declaration and the definition are different when they are not. You can see the warnings in its output (hint: redirect stdout to /dev/null to extract the warnings from the noise). Patch attached that adds some explicit namespace qualifiers. This removes doxygen's confusion, fixes the docs and silences the warnings. Regards, Jeremy Henty
On Sun, May 31, 2009 at 06:50:28AM +0100, Jeremy Henty wrote:
Doxygen is missing some methods because it's not smart enough about namespaces and sometimes thinks that the signatures of the declaration and the definition are different when they are not. You can see the warnings in its output (hint: redirect stdout to /dev/null to extract the warnings from the noise). Patch attached that adds some explicit namespace qualifiers. This removes doxygen's confusion, fixes the docs and silences the warnings.
It looks OK to me. Does anybody have any objection/comment? -- Cheers Jorge.-
On Sun, May 31, 2009 at 02:58:40PM -0400, Jorge Arellano Cid wrote:
On Sun, May 31, 2009 at 06:50:28AM +0100, Jeremy Henty wrote:
Doxygen is missing some methods because it's not smart enough about namespaces and sometimes thinks that the signatures of the declaration and the definition are different when they are not. You can see the warnings in its output (hint: redirect stdout to /dev/null to extract the warnings from the noise). Patch attached that adds some explicit namespace qualifiers. This removes doxygen's confusion, fixes the docs and silences the warnings.
It looks OK to me. Does anybody have any objection/comment?
+1 Cheers, Johannes
Johannes wrote:
On Sun, May 31, 2009 at 02:58:40PM -0400, Jorge Arellano Cid wrote:
On Sun, May 31, 2009 at 06:50:28AM +0100, Jeremy Henty wrote:
Doxygen is missing some methods because it's not smart enough about namespaces and sometimes thinks that the signatures of the declaration and the definition are different when they are not. You can see the warnings in its output (hint: redirect stdout to /dev/null to extract the warnings from the noise). Patch attached that adds some explicit namespace qualifiers. This removes doxygen's confusion, fixes the docs and silences the warnings.
It looks OK to me. Does anybody have any objection/comment?
+1
No objections.
On Sun, May 31, 2009 at 06:50:28AM +0100, Jeremy Henty wrote:
Doxygen is missing some methods because it's not smart enough about namespaces and sometimes thinks that the signatures of the declaration and the definition are different when they are not. You can see the warnings in its output (hint: redirect stdout to /dev/null to extract the warnings from the noise). Patch attached that adds some explicit namespace qualifiers. This removes doxygen's confusion, fixes the docs and silences the warnings.
Committed! -- Cheers Jorge.-
participants (4)
-
corvid＠lavabit.com -
jcid＠dillo.org -
Johannes.Hofmann＠gmx.de -
onepoint＠starurchin.org