postgres.git / summary / log / commit / refs

commit    8cd61d288adfabe1b7f83359eb0abd27382eec08
Author:   Peter Eisentraut <peter@eisentraut.org>
Date:     Thu Jun 30 18:10:13 2022 +0000
First released in: 16.0

    Avoid unnecessary MemSet call
    
    The variable in question was changed from a struct to a pointer some
    time ago (77947c51c08).  Using MemSet to zero it still works but is
    obviously unidiomatic and confusing, so change it to a straight
    assignment.
    
    Author: Ranier Vilela <ranier.vf@gmail.com>
    Discussion: https://www.postgresql.org/message-id/flat/CAEudQApCeq4JjW1BdnwU=m=-DvG5WyUik0Yfn3p6UNphiHjj+w@mail.gmail.com


 src/backend/utils/cache/relcache.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c
index 0e8fda97f86..f502df91dca 100644
--- a/src/backend/utils/cache/relcache.c
+++ b/src/backend/utils/cache/relcache.c
@@ -6258,7 +6258,7 @@ load_relcache_init_file(bool shared)
 		rel->rd_firstRelfilenodeSubid = InvalidSubTransactionId;
 		rel->rd_droppedSubid = InvalidSubTransactionId;
 		rel->rd_amcache = NULL;
-		MemSet(&rel->pgstat_info, 0, sizeof(rel->pgstat_info));
+		rel->pgstat_info = NULL;
 
 		/*
 		 * Recompute lock and physical addressing info.  This is needed in


[parent: d31d30973a19]