diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-11-29 00:09:17 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2017-11-29 00:13:45 -0300 |
| commit | 3848d21673e9dcb42d4bc1353043d7c7d6f550cd (patch) | |
| tree | 45f6f80434d6cd82277375e964819a11968845cd /src | |
| parent | 414cd434ff681e5f499803458eae9d5bb32372a9 (diff) | |
| download | postgresql-3848d21673e9dcb42d4bc1353043d7c7d6f550cd.tar.gz postgresql-3848d21673e9dcb42d4bc1353043d7c7d6f550cd.zip | |
Make memset() use sizeof() rather than re-compute size
This is simpler and more closely follows overwhelming precedent.
Report and patch by Mark Dilger.
Discussion: https://postgr.es/m/9A68FB88-5F45-4848-9926-8586E2D777D1@gmail.com
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/statistics/extended_stats.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/statistics/extended_stats.c b/src/backend/statistics/extended_stats.c index eeed56ff0aa..26c2aedd36c 100644 --- a/src/backend/statistics/extended_stats.c +++ b/src/backend/statistics/extended_stats.c @@ -301,9 +301,9 @@ statext_store(Relation pg_stext, Oid statOid, bool nulls[Natts_pg_statistic_ext]; bool replaces[Natts_pg_statistic_ext]; - memset(nulls, 1, Natts_pg_statistic_ext * sizeof(bool)); - memset(replaces, 0, Natts_pg_statistic_ext * sizeof(bool)); - memset(values, 0, Natts_pg_statistic_ext * sizeof(Datum)); + memset(nulls, true, sizeof(nulls)); + memset(replaces, false, sizeof(replaces)); + memset(values, 0, sizeof(values)); /* * Construct a new pg_statistic_ext tuple, replacing the calculated stats. |