From d429ebe347a5f1404cc3c50234e1c93f8db2bb53 Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Thu, 9 Feb 2012 12:22:21 -0500 Subject: [PATCH] Add a comment to AdjustIntervalForTypmod to reduce chance of future bugs. It's not entirely evident how the logic here relates to the interval_transform function, so let's clue people in that they need to check that if the rules change. --- src/backend/utils/adt/timestamp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/backend/utils/adt/timestamp.c b/src/backend/utils/adt/timestamp.c index 12af7f69f6..db434dcb3c 100644 --- a/src/backend/utils/adt/timestamp.c +++ b/src/backend/utils/adt/timestamp.c @@ -1069,7 +1069,8 @@ AdjustIntervalForTypmod(Interval *interval, int32 typmod) * nonzero "month" field. However that seems a bit pointless when we * can't do it consistently. (We cannot enforce a range limit on the * highest expected field, since we do not have any equivalent of - * SQL's .) + * SQL's .) If we ever decide to + * revisit this, interval_transform will likely requite adjusting. * * Note: before PG 8.4 we interpreted a limited set of fields as * actually causing a "modulo" operation on a given value, potentially