Noticed while reviewing the previous patch; thankfully, there
are no violations.
* cfg.mk (useless_free_options): Add xmlFreeNode.
---
> + if (def->metadata)
> + xmlFreeNode(def->metadata);
Useless use of if before free, since xmlFreeNode is documented as a
no-op on NULL. I'll update our list of function names to include
xmlFreeNode(), so that 'make syntax-check' will prevent this in the
future, as a followup patch.
Pushing under the trivial rule.
cfg.mk | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/cfg.mk b/cfg.mk
index 20a085e..d853caf 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -171,6 +171,7 @@ useless_free_options = \
--name=xmlBufferFree \
--name=xmlFree \
--name=xmlFreeDoc \
+ --name=xmlFreeNode \
--name=xmlXPathFreeContext \
--name=xmlXPathFreeObject
--
1.7.7.5