[tz] [PROPOSED 4/5] Remove no-longer-needed ATTRIBUTE_PURE

Paul Eggert eggert at cs.ucla.edu
Wed May 24 05:19:23 UTC 2017


For these static functions, modern GCC is smart enough to purity
on its own.
* localtime.c (getzname, getqzname):
* zic.c (memcheck, ciequal, itsabbr, byword):
Remove ATTRIBUTE_PURE.
---
 localtime.c | 4 ++--
 zic.c       | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/localtime.c b/localtime.c
index a951cab..8f103aa 100644
--- a/localtime.c
+++ b/localtime.c
@@ -737,7 +737,7 @@ static const int	year_lengths[2] = {
 ** character.
 */
 
-static const char * ATTRIBUTE_PURE
+static const char *
 getzname(register const char *strp)
 {
 	register char	c;
@@ -757,7 +757,7 @@ getzname(register const char *strp)
 ** We don't do any checking here; checking is done later in common-case code.
 */
 
-static const char * ATTRIBUTE_PURE
+static const char *
 getqzname(register const char *strp, const int delim)
 {
 	register int	c;
diff --git a/zic.c b/zic.c
index 486631b..f57d346 100644
--- a/zic.c
+++ b/zic.c
@@ -415,7 +415,7 @@ strdup(char const *str)
 }
 #endif
 
-static ATTRIBUTE_PURE void *
+static void *
 memcheck(void *ptr)
 {
 	if (ptr == NULL)
@@ -2878,7 +2878,7 @@ lowerit(char a)
 }
 
 /* case-insensitive equality */
-static ATTRIBUTE_PURE bool
+static bool
 ciequal(register const char *ap, register const char *bp)
 {
 	while (lowerit(*ap) == lowerit(*bp++))
@@ -2887,7 +2887,7 @@ ciequal(register const char *ap, register const char *bp)
 	return false;
 }
 
-static ATTRIBUTE_PURE bool
+static bool
 itsabbr(register const char *abbr, register const char *word)
 {
 	if (lowerit(*abbr) != lowerit(*word))
@@ -2901,7 +2901,7 @@ itsabbr(register const char *abbr, register const char *word)
 	return true;
 }
 
-static ATTRIBUTE_PURE const struct lookup *
+static const struct lookup *
 byword(const char *word, const struct lookup *table)
 {
 	register const struct lookup *	foundlp;
-- 
2.9.4



More information about the tz mailing list