From fd0625c7a9c679c0c1e896014b8f49a489c3a245 Mon Sep 17 00:00:00 2001 From: Michael Paquier Date: Wed, 8 Sep 2021 09:44:04 +0900 Subject: 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 --- src/backend/utils/adt/ruleutils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/backend/utils/adt/ruleutils.c') 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 */ -- cgit v1.2.3