Cleanup: get rid of req_null

Author: asfernandes

src/dsql/AggNodes.cpp

@@ -388,7 +388,7 @@ bool AggNode::aggPass(thread_db* tdbb, Request* request) const
 	if (arg)
 	{
 		desc = EVL_expr(tdbb, request, arg);
-		if (request->req_flags & req_null)
+		if (!desc)
 			return false;
 
 		if (distinct)
@@ -597,7 +597,7 @@ void AnyValueAggNode::aggPass(thread_db* tdbb, Request* request, dsc* desc) cons
 	{
 		const auto argValue = EVL_expr(tdbb, request, arg);
 
-		if (!(request->req_flags & req_null))
+		if (!argValue)
 			EVL_make_value(tdbb, argValue, impure);
 	}
 
@@ -1037,7 +1037,7 @@ void ListAggNode::aggPass(thread_db* tdbb, Request* request, dsc* desc) const
 	{
 		const dsc* const delimiterDesc = EVL_expr(tdbb, request, delimiter);
 
-		if (request->req_flags & req_null)
+		if (!delimiterDesc)
 		{
 			// Mark the result as NULL.
 			impure->vlu_desc.dsc_dtype = 0;
@@ -2031,11 +2031,11 @@ bool CorrAggNode::aggPass(thread_db* tdbb, Request* request) const
 	dsc* desc2 = NULL;
 
 	desc = EVL_expr(tdbb, request, arg);
-	if (request->req_flags & req_null)
+	if (!desc)
 		return false;
 
 	desc2 = EVL_expr(tdbb, request, arg2);
-	if (request->req_flags & req_null)
+	if (!desc2)
 		return false;
 
 	++impure->vlux_count;
@@ -2307,11 +2307,11 @@ bool RegrAggNode::aggPass(thread_db* tdbb, Request* request) const
 	dsc* desc2 = NULL;
 
 	desc = EVL_expr(tdbb, request, arg);
-	if (request->req_flags & req_null)
+	if (!desc)
 		return false;
 
 	desc2 = EVL_expr(tdbb, request, arg2);
-	if (request->req_flags & req_null)
+	if (!desc2)
 		return false;
 
 	++impure->vlux_count;
@@ -2559,12 +2559,10 @@ void RegrCountAggNode::aggInit(thread_db* tdbb, Request* request) const
 
 bool RegrCountAggNode::aggPass(thread_db* tdbb, Request* request) const
 {
-	EVL_expr(tdbb, request, arg);
-	if (request->req_flags & req_null)
+	if (!EVL_expr(tdbb, request, arg))
 		return false;
 
-	EVL_expr(tdbb, request, arg2);
-	if (request->req_flags & req_null)
+	if (!EVL_expr(tdbb, request, arg2))
 		return false;
 
 	impure_value_ex* impure = request->getImpure<impure_value_ex>(impureOffset);