Mark Dilger <[EMAIL PROTECTED]> writes: > justify_days(justify_hours(...)) fixes *everything* in the most recently > submitted patch, regardless of the convoluted case you invent. There is no > data > for which it won't work.
If so, one function or the other is cheating. Per discussion, justify_hours must never touch months, and I don't believe that justify_days should touch seconds either. The proposed justify_interval function should have a result different from successive application of the two existing functions, because it will ensure that all three fields have similar signs whereas separate use of the two functions can't promise that in corner cases. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 2: Don't 'kill -9' the postmaster