From 89a368418c56125f79f46a81229e824d519f9718 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sat, 20 Nov 2010 15:07:16 -0500 Subject: Further cleanup of indxpath logic related to IndexOptInfo.opfamily array. We no longer need the terminating zero entry in opfamily[], so get rid of it. Also replace assorted ad-hoc looping logic with simple for and foreach constructs. This code is now noticeably more readable than it was an hour ago; credit to Robert for seeing that it could be simplified. --- src/include/nodes/relation.h | 3 --- 1 file changed, 3 deletions(-) (limited to 'src/include') diff --git a/src/include/nodes/relation.h b/src/include/nodes/relation.h index 81126a23665..677134dcec9 100644 --- a/src/include/nodes/relation.h +++ b/src/include/nodes/relation.h @@ -427,9 +427,6 @@ typedef struct RelOptInfo * * opfamily[], indexkeys[], opcintype[], fwdsortop[], revsortop[], * and nulls_first[] each have ncolumns entries. - * Note: for historical reasons, the opfamily array has an extra entry - * that is always zero. Some code scans until it sees a zero entry, - * rather than looking at ncolumns. * * Zeroes in the indexkeys[] array indicate index columns that are * expressions; there is one element in indexprs for each such column. -- cgit v1.2.3