postgres.git / summary / log / commit / refs

commit    4bc4cfe3bd186b4a1d1b01279bfd0e6ab11268b2
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Tue Aug 16 20:14:16 2016 +0000

    Suppress -Wunused-result warning for strtol().
    
    I'm not sure which bozo thought it's a problem to use strtol() only
    for its endptr result, but silence the warning using same method
    used elsewhere.
    
    Report: <f845d3a6-5328-3e2a-924f-f8e91aa2b6d2@2ndquadrant.com>


 src/backend/utils/adt/dbsize.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/backend/utils/adt/dbsize.c b/src/backend/utils/adt/dbsize.c
index 0e8a82d6f4d..3167bad92b0 100644
--- a/src/backend/utils/adt/dbsize.c
+++ b/src/backend/utils/adt/dbsize.c
@@ -759,13 +759,15 @@ pg_size_bytes(PG_FUNCTION_ARGS)
 	/* Part (4): optional exponent */
 	if (*endptr == 'e' || *endptr == 'E')
 	{
+		long		exponent;
 		char	   *cp;
 
 		/*
 		 * Note we might one day support EB units, so if what follows 'E'
 		 * isn't a number, just treat it all as a unit to be parsed.
 		 */
-		(void) strtol(endptr + 1, &cp, 10);
+		exponent = strtol(endptr + 1, &cp, 10);
+		(void) exponent;		/* Silence -Wunused-result warnings */
 		if (cp > endptr + 1)
 			endptr = cp;
 	}


[parent: 7f61fd10ceb7]