Remove useless check for NULL subexpression.

tglsfdc · tglsfdc · commit 131926a52da0 · 2015-06-30T12:53:54.000-04:00
Coverity rightly gripes that it's silly to have a test here when
the adjacent ExecEvalExpr() would choke on a NULL expression pointer.

Petr Jelinek
diff --git a/src/backend/access/tablesample/tablesample.c b/src/backend/access/tablesample/tablesample.c
@@ -103,19 +103,13 @@ tablesample_init(SampleScanState *scanstate, TableSampleClause *tablesample)
 		fcinfo.argnull[1] = false;
 	}
 
-	/* Rest of the arguments come from user.  */
+	/* Rest of the arguments come from user. */
 	i = 2;
 	foreach(arg, args)
 	{
 		Expr	   *argexpr = (Expr *) lfirst(arg);
 		ExprState  *argstate = ExecInitExpr(argexpr, (PlanState *) scanstate);
 
-		if (argstate == NULL)
-		{
-			fcinfo.argnull[i] = true;
-			fcinfo.arg[i] = (Datum) 0;;
-		}
-
 		fcinfo.arg[i] = ExecEvalExpr(argstate, econtext,
 									 &fcinfo.argnull[i], NULL);
 		i++;
