Re: [HACKERS] workaround for buggy strtod is not necessary
Bruce Momjian napsal(a): OK, great information, updated comment is: /* * Many versions of Solaris have broken strtod() --- see bug #4751182. * This has been fixed in current versions of Solaris: * * http://sunsolve.sun.com/search/document.do?assetkey=1-21-108993-62-1&searchclause=108993-62 * http://sunsolve.sun.com/search/document.do?assetkey=1-21-112874-34-1&searchclause=112874-34 * * However, many people might not have patched versions, so * still use our own fix for the buggy version. */ Excellent, thanks Bruce Zdenek ---(end of broadcast)--- TIP 6: explain analyze is your friend
Re: [HACKERS] workaround for buggy strtod is not necessary
Zdenek Kotala wrote: > Bruce Momjian wrote: > > Zdenek Kotala wrote: > >> Solaris had broken strtod function when parse "Inf" and "Nan". See > >> solaris.h. This bug has been fixed for all current versions of Solaris ( > >> 8, 9, 10). See > >> > >> http://sunsolve.sun.com/search/document.do?assetkey=1-21-108993-62-1&searchclause=108993-62 > >> http://sunsolve.sun.com/search/document.do?assetkey=1-21-112874-34-1&searchclause=112874-34 > >> > >> > >> Workaround is not necessary now, but I'm not sure if workaround removing > >> is good idea. There are some possible variants: > >> > >> > >> 1) keep workaround in the code for compatibility with unpatched system > >> and adjust comment in solaris.h > >> > >> 2) remove workaround and mention required Solaris patch in the > >> Solaris.FAQ document > >> > >> 3) add some test into configure script and use workaround only on > >> unpatched system. (It should be nightmare for packagers) > > > > #1. What do you want to change about the existing comment? > > > > /* > > * Many versions of Solaris have broken strtod() --- see bug #4751182. > > * For the moment we just assume they all do; it's probably not worth > > * the trouble to add a configure test for this. > > */ > > > > /* Many versions of Solaris had broken strtod() -- see bug #4751182. > Now, it is fixed in recent versions of Solaris (8 and above -- see patch > 108993-62 and 112874-34). However, there should be many unpatched > system. For the moment we just assume they all do; it's probably not > worth the trouble to add a configure test for this. */ > > > It is only update that it is fixed and why workaround is not removed. OK, great information, updated comment is: /* * Many versions of Solaris have broken strtod() --- see bug #4751182. * This has been fixed in current versions of Solaris: * * http://sunsolve.sun.com/search/document.do?assetkey=1-21-108993-62-1&searchclause=108993-62 * http://sunsolve.sun.com/search/document.do?assetkey=1-21-112874-34-1&searchclause=112874-34 * * However, many people might not have patched versions, so * still use our own fix for the buggy version. */ -- Bruce Momjian [EMAIL PROTECTED] EnterpriseDBhttp://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---(end of broadcast)--- TIP 1: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
Re: [HACKERS] workaround for buggy strtod is not necessary
Bruce Momjian wrote: Zdenek Kotala wrote: Solaris had broken strtod function when parse "Inf" and "Nan". See solaris.h. This bug has been fixed for all current versions of Solaris ( 8, 9, 10). See http://sunsolve.sun.com/search/document.do?assetkey=1-21-108993-62-1&searchclause=108993-62 http://sunsolve.sun.com/search/document.do?assetkey=1-21-112874-34-1&searchclause=112874-34 Workaround is not necessary now, but I'm not sure if workaround removing is good idea. There are some possible variants: 1) keep workaround in the code for compatibility with unpatched system and adjust comment in solaris.h 2) remove workaround and mention required Solaris patch in the Solaris.FAQ document 3) add some test into configure script and use workaround only on unpatched system. (It should be nightmare for packagers) #1. What do you want to change about the existing comment? /* * Many versions of Solaris have broken strtod() --- see bug #4751182. * For the moment we just assume they all do; it's probably not worth * the trouble to add a configure test for this. */ /* Many versions of Solaris had broken strtod() -- see bug #4751182. Now, it is fixed in recent versions of Solaris (8 and above -- see patch 108993-62 and 112874-34). However, there should be many unpatched system. For the moment we just assume they all do; it's probably not worth the trouble to add a configure test for this. */ It is only update that it is fixed and why workaround is not removed. Zdenek ---(end of broadcast)--- TIP 2: Don't 'kill -9' the postmaster
Re: [HACKERS] workaround for buggy strtod is not necessary
Zdenek Kotala wrote: > > Solaris had broken strtod function when parse "Inf" and "Nan". See > solaris.h. This bug has been fixed for all current versions of Solaris ( > 8, 9, 10). See > > http://sunsolve.sun.com/search/document.do?assetkey=1-21-108993-62-1&searchclause=108993-62 > http://sunsolve.sun.com/search/document.do?assetkey=1-21-112874-34-1&searchclause=112874-34 > > > Workaround is not necessary now, but I'm not sure if workaround removing > is good idea. There are some possible variants: > > > 1) keep workaround in the code for compatibility with unpatched system > and adjust comment in solaris.h > > 2) remove workaround and mention required Solaris patch in the > Solaris.FAQ document > > 3) add some test into configure script and use workaround only on > unpatched system. (It should be nightmare for packagers) #1. What do you want to change about the existing comment? /* * Many versions of Solaris have broken strtod() --- see bug #4751182. * For the moment we just assume they all do; it's probably not worth * the trouble to add a configure test for this. */ -- Bruce Momjian [EMAIL PROTECTED] EnterpriseDBhttp://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---(end of broadcast)--- TIP 4: Have you searched our list archives? http://archives.postgresql.org