diff options
| author | Michael Paquier <michael@paquier.xyz> | 2021-09-08 09:44:04 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2021-09-08 09:44:04 +0900 |
| commit | fd0625c7a9c679c0c1e896014b8f49a489c3a245 (patch) | |
| tree | cacddafcd64a4f00d87f6551b29937f9e5ced415 /src/backend/executor | |
| parent | d6c916f020e291b45563d4e76a649e9364cb6f2d (diff) | |
| download | postgresql-fd0625c7a9c679c0c1e896014b8f49a489c3a245.tar.gz postgresql-fd0625c7a9c679c0c1e896014b8f49a489c3a245.zip | |
Clean up some code using "(expr) ? true : false"
All the code paths simplified here were already using a boolean or used
an expression that led to zero or one, making the extra bits
unnecessary.
Author: Justin Pryzby
Reviewed-by: Tom Lane, Michael Paquier, Peter Smith
Discussion: https://postgr.es/m/20210428182936.GE27406@telsasoft.com
Diffstat (limited to 'src/backend/executor')
| -rw-r--r-- | src/backend/executor/spi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/executor/spi.c b/src/backend/executor/spi.c index a8d7fe6dabf..a5aec7ba7db 100644 --- a/src/backend/executor/spi.c +++ b/src/backend/executor/spi.c @@ -1037,7 +1037,7 @@ SPI_modifytuple(Relation rel, HeapTuple tuple, int natts, int *attnum, if (attnum[i] <= 0 || attnum[i] > numberOfAttributes) break; v[attnum[i] - 1] = Values[i]; - n[attnum[i] - 1] = (Nulls && Nulls[i] == 'n') ? true : false; + n[attnum[i] - 1] = (Nulls && Nulls[i] == 'n'); } if (i == natts) /* no errors in *attnum */ |