From a412749812823b03763b747080b57a3d316f33c4 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Sat, 23 Dec 2000 04:05:31 +0000 Subject: [PATCH] Replace overly-cute coding with code that (a) has defined behavior according to the ANSI C spec, (b) gets the boundary conditions right, and (c) is about a third as long and three times more intelligible. --- src/backend/utils/adt/formatting.c | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/src/backend/utils/adt/formatting.c b/src/backend/utils/adt/formatting.c index 2d540d38d8..789b7b9513 100644 --- a/src/backend/utils/adt/formatting.c +++ b/src/backend/utils/adt/formatting.c @@ -1,7 +1,7 @@ /* ----------------------------------------------------------------------- * formatting.c * - * $Header: /cvsroot/pgsql/src/backend/utils/adt/formatting.c,v 1.27 2000/12/15 19:15:09 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/adt/formatting.c,v 1.28 2000/12/23 04:05:31 tgl Exp $ * * * Portions Copyright (c) 1999-2000, PostgreSQL, Inc @@ -2775,16 +2775,14 @@ to_timestamp(PG_FUNCTION_ARGS) #endif if (tmfc->ssss) { - int x; - - if (tmfc->ssss > 3600) - tm->tm_sec = x - ((tm->tm_min = (x = tmfc->ssss - - ((tm->tm_hour= tmfc->ssss / 3600) * 3600)) / 60) * 60); - else if (tmfc->ssss > 60) - tm->tm_sec = tmfc->ssss - ((tm->tm_min = tmfc->ssss / 60) * 60); - else - tm->tm_sec = tmfc->ssss; - } + int x = tmfc->ssss; + + tm->tm_hour = x / 3600; + x %= 3600; + tm->tm_min = x / 60; + x %= 60; + tm->tm_sec = x; + } if (tmfc->cc) tm->tm_year = (tmfc->cc-1) * 100; -- 2.11.0