Received: from localhost (maia-1.hub.org [200.46.204.191]) by postgresql.org (Postfix) with ESMTP id B29699F938C; Wed, 27 Dec 2006 17:42:27 -0400 (AST) Received: from postgresql.org ([200.46.204.71]) by localhost (mx1.hub.org [200.46.204.191]) (amavisd-new, port 10024) with ESMTP id 86578-05; Wed, 27 Dec 2006 17:41:42 -0400 (AST) X-Greylist: from auto-whitelisted by SQLgrey-1.7.4 X-Greylist: from auto-whitelisted by SQLgrey-1.7.4 Received: from lists.commandprompt.com (unknown [207.173.203.254]) by postgresql.org (Postfix) with ESMTP id E343B9FA097; Wed, 27 Dec 2006 17:41:41 -0400 (AST) Received: from [192.168.1.50] (or-67-76-146-141.sta.embarqhsd.net [67.76.146.141]) (authenticated bits=0) by lists.commandprompt.com (8.13.7/8.13.6) with ESMTP id kBRLfYxE019587 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NO); Wed, 27 Dec 2006 13:41:35 -0800 Subject: Re: [BUGS] BUG #2846: inconsistent and confusing From: "Joshua D. Drake" To: Bruce Momjian Cc: Tom Lane , pgsql-hackers@postgresql.org, PostgreSQL-patches , Roman Kononov In-Reply-To: <200612272135.kBRLZXD07532@momjian.us> References: <200612272135.kBRLZXD07532@momjian.us> Content-Type: text/plain Organization: Command Prompt, Inc. Date: Wed, 27 Dec 2006 13:43:54 -0800 Message-Id: <1167255834.12075.50.camel@localhost.localdomain> Mime-Version: 1.0 X-Mailer: Evolution 2.8.1 Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV version 0.88.5, clamav-milter version 0.88.5 on projects.commandprompt.com X-Virus-Status: Clean X-Greylist: Sender succeded SMTP AUTH authentication, not delayed by milter-greylist-1.6 (lists.commandprompt.com [192.168.2.159]); Wed, 27 Dec 2006 13:41:36 -0800 (PST) X-Virus-Scanned: Maia Mailguard 1.0.1 X-Archive-Number: 200612/160 X-Sequence-Number: 21836 > I get 'inf'. I am on BSD and just tested it on Fedora Core 2 and got > 'inf' too. Ubuntu Edgy 64bit on Athlon 64X2 returns inf. Joshua D. Drake > > > A slightly less radical proposal is to reject only the case where > > isinf(result) and neither input isinf(); and perhaps likewise with > > respect to NaNs. > > Uh, that's what the patch does for 'Inf': > > result = arg1 + arg2; > CheckFloat4Val(result, isinf(arg1) || isinf(arg2)); > > I didn't touch 'Nan' because that is passed around as a value just fine > --- it isn't created or tested as part of an overflow. > -- === The PostgreSQL Company: Command Prompt, Inc. === Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240 Providing the most comprehensive PostgreSQL solutions since 1997 http://www.commandprompt.com/ Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate