Merge pull request duckdb#12924 from ZhangHuiGui/simplify-and-remove-…

…useless-codes

Minor refactor for window_executor

src/execution/window_executor.cpp

@@ -309,7 +309,7 @@ static idx_t FindOrderedRangeBound(const WindowInputColumn &over, const OrderTyp
 
 struct WindowBoundariesState {
 	static inline bool IsScalar(const unique_ptr<Expression> &expr) {
-		return expr ? expr->IsScalar() : true;
+		return !expr || expr->IsScalar();
 	}
 
 	static inline bool BoundaryNeedsPeer(const WindowBoundary &boundary) {
@@ -1121,8 +1121,6 @@ class WindowAggregateExecutorLocalState : public WindowExecutorBoundsState {
 public:
 	// state of aggregator
 	unique_ptr<WindowAggregatorState> aggregator_state;
-
-	void NextRank(idx_t partition_begin, idx_t peer_begin, idx_t row_idx);
 };
 
 unique_ptr<WindowExecutorLocalState>