Apply tidy fixes

Author: ila

src/core/plan_rewrite.cpp

@@ -1275,16 +1275,16 @@ static void RewriteLeftJoinMatchCount(ClientContext &context, Binder &binder, un
 	// the hidden binding only to the top projection leaves LPTS with a reference to a
 	// column that no child CTE emits.
 	auto agg_bindings = agg_search->GetColumnBindings();
-	auto agg_types = agg_search->types;
+	const auto &agg_types = agg_search->types;
 	idx_t group_count = agg.groups.size();
 
 	ColumnBinding match_binding = agg_bindings[group_count + match_count_idx];
-	LogicalType match_type = agg_types[group_count + match_count_idx];
+	const LogicalType &match_type = agg_types[group_count + match_count_idx];
 	PropagateHiddenBindingThroughProjectionPath(projection_path, match_binding, match_type, openivm::MATCH_COUNT_COL);
 
 	if (is_full_outer) {
 		ColumnBinding right_match_binding = agg_bindings[group_count + right_match_count_idx];
-		LogicalType right_match_type = agg_types[group_count + right_match_count_idx];
+		const LogicalType &right_match_type = agg_types[group_count + right_match_count_idx];
 		PropagateHiddenBindingThroughProjectionPath(projection_path, right_match_binding, right_match_type,
 		                                            openivm::RIGHT_MATCH_COUNT_COL);
 	}

src/delta/operators/delim_join.cpp

@@ -114,8 +114,7 @@ static void AppendMultiplicityBindingsToJoinProjectionMaps(LogicalOperator &op,
 					if (projected_idx >= child_bindings.size()) {
 						throw InternalException(
 						    "DeltaDelimJoin: projection map index %llu out of bounds for child %llu with %llu bindings",
-						    (unsigned long long)projected_idx, (unsigned long long)child_idx,
-						    (unsigned long long)child_bindings.size());
+						    (idx_t)projected_idx, (idx_t)child_idx, (idx_t)child_bindings.size());
 					}
 				}
 				for (idx_t binding_idx = 0; binding_idx < child_bindings.size(); binding_idx++) {
@@ -270,8 +269,7 @@ static ColumnBinding FindOutputBinding(const vector<ColumnBinding> &term_binding
 		candidates += to_string(binding.table_index) + ":" + to_string(binding.column_index);
 	}
 	throw InternalException("DeltaDelimJoin: original output binding %llu:%llu not found in rewritten term [%s]",
-	                        (unsigned long long)target.table_index, (unsigned long long)target.column_index,
-	                        candidates.c_str());
+	                        (idx_t)target.table_index, (idx_t)target.column_index, candidates.c_str());
 }
 
 static bool ReplaceDelimGets(ClientContext &context, unique_ptr<LogicalOperator> &node,

src/delta/operators/join.cpp

@@ -405,7 +405,7 @@ void AppendMultiplicityToAncestorProjectionMaps(unique_ptr<LogicalOperator> &ter
 	for (size_t depth = 0; depth < leaf_path.size(); depth++) {
 		if (leaf_path[depth] >= node->children.size()) {
 			throw InternalException("%s: leaf path child %llu out of bounds at depth %llu", context_label,
-			                        (unsigned long long)leaf_path[depth], (unsigned long long)depth);
+			                        (idx_t)leaf_path[depth], (idx_t)depth);
 		}
 		ancestors.push_back(node);
 		node = node->children[leaf_path[depth]].get();
@@ -425,8 +425,7 @@ void AppendMultiplicityToAncestorProjectionMaps(unique_ptr<LogicalOperator> &ter
 					if (projected_idx >= child_bindings.size()) {
 						throw InternalException(
 						    "%s: projection map index %llu out of bounds for child %llu with %llu bindings",
-						    context_label, (unsigned long long)projected_idx, (unsigned long long)child_side,
-						    (unsigned long long)child_bindings.size());
+						    context_label, (idx_t)projected_idx, (idx_t)child_side, (idx_t)child_bindings.size());
 					}
 				}
 				idx_t mul_idx = DConstants::INVALID_INDEX;