From: Rhodri James <rhodri@kynesim.co.uk>
Date: Mon, 19 Jun 2017 13:42:03 +0000 (+0100)
Subject: Comment change: exclude sameName from coverage
X-Git-Tag: R_2_2_3~22^2~24
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7979e831ed09586b3f0a1480239e232f1f2914df;p=libexpat

Comment change: exclude sameName from coverage
---

diff --git a/expat/lib/xmltok_impl.c b/expat/lib/xmltok_impl.c
index 1e7f980d..16547237 100644
--- a/expat/lib/xmltok_impl.c
+++ b/expat/lib/xmltok_impl.c
@@ -1626,6 +1626,14 @@ PREFIX(predefinedEntityName)(const ENCODING *UNUSED_P(enc), const char *ptr,
   return 0;
 }
 
+/* This function does not appear to be called from anywhere within the
+ * library code.  It is used via the macro XmlSameName(), which is
+ * defined but never used.  Since it appears in the encoding function
+ * table, removing it is not a thing to be undertaken lightly.  For
+ * the moment, we simply exclude it from coverage tests.
+ *
+ * LCOV_EXCL_START
+ */
 static int PTRCALL
 PREFIX(sameName)(const ENCODING *enc, const char *ptr1, const char *ptr2)
 {
@@ -1689,6 +1697,7 @@ PREFIX(sameName)(const ENCODING *enc, const char *ptr1, const char *ptr2)
   }
   /* not reached */
 }
+/* LCOV_EXCL_STOP */
 
 static int PTRCALL
 PREFIX(nameMatchesAscii)(const ENCODING *UNUSED_P(enc), const char *ptr1,