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/utils/adt/ruleutils.c | |
| 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/utils/adt/ruleutils.c')
| -rw-r--r-- | src/backend/utils/adt/ruleutils.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index 8ff4e5dc07b..1fb0b7b0984 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -8005,14 +8005,14 @@ isSimpleNode(Node *node, Node *parentNode, int prettyFlags) * appears simple since . has top precedence, unless parent is * T_FieldSelect itself! */ - return (IsA(parentNode, FieldSelect) ? false : true); + return !IsA(parentNode, FieldSelect); case T_FieldStore: /* * treat like FieldSelect (probably doesn't matter) */ - return (IsA(parentNode, FieldStore) ? false : true); + return !IsA(parentNode, FieldStore); case T_CoerceToDomain: /* maybe simple, check args */ |