Corrected according to @dyemanov

Author: sim1984

builds/win32/y.txt

@@ -0,0 +1,2 @@
+y.y:6864: $$ is untyped
+y.y:6865: $$ is untyped

builds/win32/y_tab.c

@@ -0,0 +1,26 @@
+
+//
+// @(#)btyaccpar, based on byacc 1.8 (Berkeley)
+// Parser skeleton modified for use in the Firebird project by Nickolay Samofatov
+//
+#define YYBTYACC 1
+
+#include "firebird.h"
+
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <optional>
+#include "../dsql/Nodes.h"
+#include "../dsql/AggNodes.h"
+#include "../dsql/DdlNodes.h"
+#include "../dsql/BoolNodes.h"
+#include "../dsql/ExprNodes.h"
+#include "../dsql/PackageNodes.h"
+#include "../dsql/StmtNodes.h"
+#include "../dsql/WinNodes.h"
+#include "../jrd/RecordSourceNodes.h"
+#include "../common/classes/TriState.h"
+#include "gen/parse.h"
+#include "../dsql/Parser.h"
+

builds/win32/y_tab.h

@@ -6851,15 +6851,15 @@ table_value_function_gen_series
 
 %type <valueListNode> table_value_function_gen_series_arg_list
 table_value_function_gen_series_arg_list
-	: value ',' value step_opt
+	: value ',' value gen_series_step_opt
 		{
 			$$ = newNode<ValueListNode>($1);
 			$$->add($3);
 			$$->add($4);
 		}
 	;
 
-%type <valueExprNode> step_opt
+%type <valueExprNode> gen_series_step_opt
 step_opt
 	: /* nothing */		{ $$ = MAKE_const_sint64(1, 0); }
 	| ',' value			{ $$ = $2; }

src/dsql/parse.y

@@ -384,26 +384,17 @@ void GenSeriesFunctionScan::internalOpen(thread_db* tdbb) const
 	auto startItem = m_inputList->items[GEN_SERIES_INDEX_START];
 	const auto startDesc = EVL_expr(tdbb, request, startItem);
 	if (startDesc == nullptr)
-	{
-		rpb->rpb_number.setValid(false);
 		return;
-	}
 
 	auto finishItem = m_inputList->items[GEN_SERIES_INDEX_FINISH];
 	const auto finishDesc = EVL_expr(tdbb, request, finishItem);
 	if (finishDesc == nullptr)
-	{
-		rpb->rpb_number.setValid(false);
 		return;
-	}
 
 	auto stepItem = m_inputList->items[GEN_SERIES_INDEX_STEP];
 	const auto stepDesc = EVL_expr(tdbb, request, stepItem);
 	if (stepDesc == nullptr)
-	{
-		rpb->rpb_number.setValid(false);
 		return;
-	}
 
 	const auto impure = request->getImpure<Impure>(m_impure);
 	impure->m_recordBuffer = nullptr;
@@ -425,10 +416,7 @@ void GenSeriesFunctionScan::internalOpen(thread_db* tdbb) const
 		// validate parameter value
 		if (((step > 0) && (start > finish)) ||
 			((step < 0) && (start < finish)))
-		{
-			rpb->rpb_number.setValid(false);
 			return;
-		}
 
 		impure->m_start.vlu_int64 = start;
 		impure->m_finish.vlu_int64 = finish;
@@ -447,10 +435,7 @@ void GenSeriesFunctionScan::internalOpen(thread_db* tdbb) const
 		// validate parameter value
 		if (((step.sign() > 0) && (start.compare(finish) > 0)) ||
 			((step.sign() < 0) && (start.compare(finish) < 0)))
-		{
-			rpb->rpb_number.setValid(false);
 			return;
-		}
 
 		impure->m_start.vlu_int128 = start;
 		impure->m_finish.vlu_int128 = finish;