Обсуждение: pgsql: Replace many MemSet calls with struct initialization

Поиск
Список
Период
Сортировка

pgsql: Replace many MemSet calls with struct initialization

От
Peter Eisentraut
Дата:
Replace many MemSet calls with struct initialization

This replaces all MemSet() calls with struct initialization where that
is easily and obviously possible.  (For example, some cases have to
worry about padding bits, so I left those.)

(The same could be done with appropriate memset() calls, but this
patch is part of an effort to phase out MemSet(), so it doesn't touch
memset() calls.)

Reviewed-by: Ranier Vilela <ranier.vf@gmail.com>
Reviewed-by: Alvaro Herrera <alvherre@alvh.no-ip.org>
Discussion: https://www.postgresql.org/message-id/9847b13c-b785-f4e2-75c3-12ec77a3b05c@enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/9fd45870c1436b477264c0c82eb195df52bc0919

Modified Files
--------------
contrib/amcheck/verify_heapam.c                 |   6 +-
contrib/bloom/blcost.c                          |   4 +-
contrib/pageinspect/brinfuncs.c                 |   7 +-
contrib/pageinspect/hashfuncs.c                 |  16 +--
contrib/pageinspect/heapfuncs.c                 |   8 +-
contrib/pg_prewarm/autoprewarm.c                |   6 +-
contrib/pg_stat_statements/pg_stat_statements.c |   7 +-
contrib/pg_visibility/pg_visibility.c           |  15 +--
contrib/pg_walinspect/pg_walinspect.c           |  25 ++--
contrib/pgstattuple/pgstatindex.c               |   3 +-
contrib/postgres_fdw/connection.c               |   7 +-
contrib/postgres_fdw/postgres_fdw.c             |   3 +-
src/backend/access/transam/twophase.c           |   6 +-
src/backend/access/transam/xlogfuncs.c          |   7 +-
src/backend/catalog/aclchk.c                    | 146 +++++++-----------------
src/backend/catalog/dependency.c                |   3 +-
src/backend/catalog/heap.c                      |  24 ++--
src/backend/catalog/index.c                     |   4 +-
src/backend/catalog/pg_attrdef.c                |   9 +-
src/backend/catalog/pg_cast.c                   |   4 +-
src/backend/catalog/pg_parameter_acl.c          |   6 +-
src/backend/catalog/pg_publication.c            |   6 +-
src/backend/commands/dbcommands.c               |  35 ++----
src/backend/commands/event_trigger.c            |  11 +-
src/backend/commands/functioncmds.c             |   7 +-
src/backend/commands/prepare.c                  |   4 +-
src/backend/commands/tablecmds.c                |  44 +++----
src/backend/commands/tablespace.c               |   4 +-
src/backend/commands/typecmds.c                 |   9 +-
src/backend/commands/user.c                     |  37 ++----
src/backend/optimizer/path/costsize.c           |   3 +-
src/backend/parser/parse_target.c               |   3 +-
src/backend/replication/logical/launcher.c      |   7 +-
src/backend/replication/walsender.c             |  17 +--
src/backend/rewrite/rewriteDefine.c             |   8 +-
src/backend/utils/adt/acl.c                     |   4 +-
src/backend/utils/adt/arrayfuncs.c              |   3 +-
src/backend/utils/adt/datetime.c                |   8 +-
src/backend/utils/adt/lockfuncs.c               |  12 +-
src/backend/utils/adt/partitionfuncs.c          |   6 +-
src/backend/utils/adt/pgstatfuncs.c             |  52 +++------
src/backend/utils/adt/selfuncs.c                |  19 +--
src/backend/utils/adt/timestamp.c               |   8 +-
src/backend/utils/mmgr/portalmem.c              |   4 +-
src/bin/pg_basebackup/pg_basebackup.c           |  13 +--
src/bin/pg_basebackup/pg_receivewal.c           |   4 +-
src/bin/pg_basebackup/walmethods.c              |   6 +-
src/common/ip.c                                 |   4 +-
src/port/snprintf.c                             |   8 +-
src/test/modules/test_predtest/test_predtest.c  |   3 +-
src/test/regress/regress.c                      |   3 +-
51 files changed, 200 insertions(+), 468 deletions(-)


Re: pgsql: Replace many MemSet calls with struct initialization

От
Justin Pryzby
Дата:
On Sat, Jul 16, 2022 at 06:53:16AM +0000, Peter Eisentraut wrote:
> Replace many MemSet calls with struct initialization

This failed
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=lapwing&dt=2022-07-16%2008%3A40%3A14

It avoids the warning here to use {{0}}.
But it doesn't seem worth it to use that just to appease the compiler.

We ran into this before here: 20211202033145.GK17618@telsasoft.com

-- 
Justin