diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-11-20 15:07:16 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-11-20 15:07:16 -0500 |
| commit | 89a368418c56125f79f46a81229e824d519f9718 (patch) | |
| tree | 20fcf3b7c087f1abe9c68b67f939547f763c5797 /src/include | |
| parent | 99bc012d5198bdfd2aa67336f645dbf887bb0ec9 (diff) | |
| download | postgresql-89a368418c56125f79f46a81229e824d519f9718.tar.gz postgresql-89a368418c56125f79f46a81229e824d519f9718.zip | |
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.
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/nodes/relation.h | 3 |
1 files changed, 0 insertions, 3 deletions
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. |