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/partitioning/partbounds.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/partitioning/partbounds.c')
| -rw-r--r-- | src/backend/partitioning/partbounds.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/partitioning/partbounds.c b/src/backend/partitioning/partbounds.c index a49e97a2251..0823fa7b1db 100644 --- a/src/backend/partitioning/partbounds.c +++ b/src/backend/partitioning/partbounds.c @@ -3206,7 +3206,7 @@ check_new_partition_bound(char *relname, Relation parent, * datums list. */ PartitionRangeDatum *datum = - list_nth(spec->upperdatums, Abs(cmpval) - 1); + list_nth(spec->upperdatums, abs(cmpval) - 1); /* * The new partition overlaps with the @@ -3232,7 +3232,7 @@ check_new_partition_bound(char *relname, Relation parent, * if we have equality, point to the first one. */ datum = cmpval == 0 ? linitial(spec->lowerdatums) : - list_nth(spec->lowerdatums, Abs(cmpval) - 1); + list_nth(spec->lowerdatums, abs(cmpval) - 1); overlap = true; overlap_location = datum->location; with = boundinfo->indexes[offset + 1]; |