From cb9d929eab5ce91f55dcf63120dd39d214699d52 Mon Sep 17 00:00:00 2001 From: Jussi Saurio Date: Fri, 7 Feb 2025 10:09:42 +0200 Subject: [PATCH] call cursor methods instead of duplicating logic --- extensions/series/src/lib.rs | 17 ++--------------- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/extensions/series/src/lib.rs b/extensions/series/src/lib.rs index ffa0871c6..ddae6aa99 100644 --- a/extensions/series/src/lib.rs +++ b/extensions/series/src/lib.rs @@ -83,24 +83,11 @@ impl VTabModule for GenerateSeriesVTab { } fn next(cursor: &mut Self::VCursor) -> ResultCode { - // Check for invalid ranges (empty series) first - if cursor.eof() { - return ResultCode::EOF; - } - - // Handle overflow - cursor.current = match cursor.current.checked_add(cursor.step) { - Some(val) => val, - None => { - return ResultCode::EOF; - } - }; - - ResultCode::OK + cursor.next() } fn eof(cursor: &Self::VCursor) -> bool { - cursor.is_invalid_range() || cursor.would_exceed() + cursor.eof() } }