Remove useless casting to same type

This removes some casts where the input already has the same type as
the type specified by the cast.  Their presence could cause risks of
hiding actual type mismatches in the future or silently discarding
qualifiers.  It also improves readability.  Same kind of idea as
7f798aca1d5 and ef8fe693606.  (This does not change all such
instances, but only those hand-picked by the author.)

Author: Bertrand Drouvot <[email protected]>
Reviewed-by: Nathan Bossart <[email protected]>
Reviewed-by: Jacob Champion <[email protected]>
Reviewed-by: Peter Eisentraut <[email protected]>
Discussion: 
https://www.postgresql.org/message-id/flat/aSQy2JawavlVlEB0%40ip-10-97-1-34.eu-west-3.compute.internal

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/4f941d432b42eccd99ba0d22e3a59c073ac2406a

Modified Files
--------------
contrib/btree_gist/btree_utils_num.c             |  4 ++--
contrib/cube/cube.c                              |  6 +++---
contrib/fuzzystrmatch/dmetaphone.c               |  2 +-
contrib/pgcrypto/mbuf.c                          |  2 +-
src/backend/access/common/indextuple.c           |  2 +-
src/backend/access/gin/gindatapage.c             |  8 ++++----
src/backend/access/gin/gininsert.c               |  2 +-
src/backend/access/hash/hash_xlog.c              |  4 ++--
src/backend/access/transam/twophase.c            |  2 +-
src/backend/catalog/aclchk.c                     |  4 ++--
src/backend/commands/tablecmds.c                 |  2 +-
src/backend/executor/execExpr.c                  |  2 +-
src/backend/executor/execExprInterp.c            |  2 +-
src/backend/executor/execPartition.c             |  2 +-
src/backend/executor/nodeTableFuncscan.c         |  2 +-
src/backend/optimizer/geqo/geqo_pool.c           |  4 ++--
src/backend/optimizer/plan/planner.c             |  4 ++--
src/backend/optimizer/util/pathnode.c            |  2 +-
src/backend/parser/analyze.c                     |  2 +-
src/backend/parser/parse_expr.c                  |  4 ++--
src/backend/port/sysv_shmem.c                    |  2 +-
src/backend/replication/walsender.c              |  2 +-
src/backend/rewrite/rewriteHandler.c             |  2 +-
src/backend/statistics/dependencies.c            |  4 ++--
src/backend/statistics/extended_stats.c          |  4 ++--
src/backend/statistics/mcv.c                     |  2 +-
src/backend/storage/aio/aio.c                    |  6 +++---
src/backend/storage/aio/method_io_uring.c        |  2 +-
src/backend/storage/ipc/waiteventset.c           |  2 +-
src/backend/storage/lmgr/predicate.c             |  2 +-
src/backend/storage/lmgr/proc.c                  |  8 ++++----
src/backend/utils/adt/arrayfuncs.c               |  4 ++--
src/backend/utils/adt/jsonfuncs.c                |  6 ++----
src/backend/utils/adt/ruleutils.c                |  2 +-
src/bin/pg_dump/pg_dump.c                        |  8 ++++----
src/common/sha1.c                                |  4 ++--
src/interfaces/ecpg/ecpglib/descriptor.c         |  2 +-
src/interfaces/ecpg/ecpglib/execute.c            | 18 ++++++++----------
src/interfaces/ecpg/ecpglib/prepare.c            |  6 +++---
src/port/win32ntdll.c                            |  2 +-
src/test/modules/test_radixtree/test_radixtree.c |  2 +-
41 files changed, 74 insertions(+), 78 deletions(-)

Reply via email to