diff options
| author | John Naylor <john.naylor@postgresql.org> | 2023-01-17 13:52:11 +0700 |
|---|---|---|
| committer | John Naylor <john.naylor@postgresql.org> | 2023-01-17 13:55:49 +0700 |
| commit | e29c5653434ebf6a88d009746d21ca3a59ddb69b (patch) | |
| tree | a07654a1c4335daa702b8c92812c19744222ffbf | |
| parent | c981d9145deae067bc67bc8f8bcd68b300ece3fe (diff) | |
| download | postgresql-e29c5653434ebf6a88d009746d21ca3a59ddb69b.tar.gz postgresql-e29c5653434ebf6a88d009746d21ca3a59ddb69b.zip | |
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
| -rw-r--r-- | src/backend/utils/adt/formatting.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/backend/utils/adt/formatting.c b/src/backend/utils/adt/formatting.c index a4b524ea3ac..f3f4db5ef60 100644 --- a/src/backend/utils/adt/formatting.c +++ b/src/backend/utils/adt/formatting.c @@ -5664,13 +5664,9 @@ NUM_processor(FormatNode *node, NUMDesc *Num, char *inout, { if (Np->sign != '-') { - if (IS_BRACKET(Np->Num) && IS_FILLMODE(Np->Num)) + if (IS_FILLMODE(Np->Num)) Np->Num->flag &= ~NUM_F_BRACKET; - if (IS_MINUS(Np->Num)) - Np->Num->flag &= ~NUM_F_MINUS; } - else if (Np->sign != '+' && IS_PLUS(Np->Num)) - Np->Num->flag &= ~NUM_F_PLUS; if (Np->sign == '+' && IS_FILLMODE(Np->Num) && IS_LSIGN(Np->Num) == false) Np->sign_wrote = true; /* needn't sign */ |