Remove dead code in formatting.c
authorJohn Naylor <john.naylor@postgresql.org>
Tue, 17 Jan 2023 06:52:11 +0000 (13:52 +0700)
committerJohn Naylor <john.naylor@postgresql.org>
Tue, 17 Jan 2023 06:55:49 +0000 (13:55 +0700)
commite29c5653434ebf6a88d009746d21ca3a59ddb69b
treea07654a1c4335daa702b8c92812c19744222ffbf
parentc981d9145deae067bc67bc8f8bcd68b300ece3fe
Remove dead code in formatting.c

Remove some code guarded by IS_MINUS() or IS_PLUS(), where the entire
stanza is inside an else-block where both of these are false. This
should slightly improve test coverage.

While at it, remove coding that apparently assumes that unsetting a
bit is so expensive that we have to first check if it's already set
in the first place.

Per Coverity report from Ranier Vilela
Analysis and review by Justin Pryzby

Discussion: https://www.postgresql.org/message-id/20221223010818.GP1153%40telsasoft.com
src/backend/utils/adt/formatting.c