diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2022-10-07 13:28:38 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2022-10-07 13:29:33 +0200 |
| commit | f14aad5169baa5e2ac25d49f1d18f9d5cb3bc7f2 (patch) | |
| tree | e26a18d755d474d8695e763ee118f48146d1bea6 /src/backend/utils/adt/datetime.c | |
| parent | 0fe954c28584169938e5c0738cfaa9930ce77577 (diff) | |
| download | postgresql-f14aad5169baa5e2ac25d49f1d18f9d5cb3bc7f2.tar.gz postgresql-f14aad5169baa5e2ac25d49f1d18f9d5cb3bc7f2.zip | |
Remove unnecessary uses of Abs()
Use C standard abs() or fabs() instead.
Reviewed-by: Zhang Mingli <zmlpostgres@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://www.postgresql.org/message-id/flat/4beb42b5-216b-bce8-d452-d924d5794c63%40enterprisedb.com
Diffstat (limited to 'src/backend/utils/adt/datetime.c')
| -rw-r--r-- | src/backend/utils/adt/datetime.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/utils/adt/datetime.c b/src/backend/utils/adt/datetime.c index 904ac2353cf..a8b025f43fa 100644 --- a/src/backend/utils/adt/datetime.c +++ b/src/backend/utils/adt/datetime.c @@ -448,14 +448,14 @@ AppendSeconds(char *cp, int sec, fsec_t fsec, int precision, bool fillzeros) Assert(precision >= 0); if (fillzeros) - cp = pg_ultostr_zeropad(cp, Abs(sec), 2); + cp = pg_ultostr_zeropad(cp, abs(sec), 2); else - cp = pg_ultostr(cp, Abs(sec)); + cp = pg_ultostr(cp, abs(sec)); /* fsec_t is just an int32 */ if (fsec != 0) { - int32 value = Abs(fsec); + int32 value = abs(fsec); char *end = &cp[precision + 1]; bool gotnonzero = false; @@ -490,7 +490,7 @@ AppendSeconds(char *cp, int sec, fsec_t fsec, int precision, bool fillzeros) * which will generate a correct answer in the minimum valid width. */ if (value) - return pg_ultostr(cp, Abs(fsec)); + return pg_ultostr(cp, abs(fsec)); return end; } |