Miguel and Ron: > Ye has little overhead, but by default terminal aplications don't need it. > TRACE is a good tool for developers, but not for released.
I respectfully but strongly disagree. First of all to clarify the bigger picture for Miguel: Thank you so much for all the great work you've been doing. This is one tiny decision I think should go the other way, but it's insignificant against all the great decisions you've been implementing. Meanwhile re HB_NO_TRACE: This flag was only implemented to remove __TRACEPRGCALLS and one other trace in hb_itemNew () __TRACEPRGCALLS was created as a In-The-Field debugging tool to find where a GPF or Hang occurs by tracing up to the point where things die. It is important that it be available in releases so it can be turned on via the function call. We ran some tests, and in 5 full seconds of initialization of a large Fivewin app, cranking tens of thousands of Prg calls, the times with defining HB_NO_TRACE or not were completely imperceptible. In several runs, Windows being what it is, the code WITH the Trace enabled (remember this only means there are "if hb_bTracePrgCalls" tests, with no actual tracing) was sometimes faster. So we will be building xHarbour for ourselves with the new #define to enable this function, but FWIW I don't think we should be changing default behaviors without being able to demonstrate any real benefit. Just my two cents. Thanks again, -- Brian Hays > -----Original Message----- > From: Miguel Angel Marchuet [mailto:miguelan...@marchuet.net] > Sent: Tuesday, March 10, 2009 1:10 PM > To: Luis Krause Mantilla > Cc: Ron Pinkas; modals...@yahoo.com.br; Xharbour-Developers List > Subject: Re: [xHarbour-developers] Upper() and codepage. > > Ye has little overhead, but by default terminal aplications don't need > it. > > TRACE is a good tool for developers, but not for released. > > I personally prefer not overhead by default the compiler. > > Best regards, > Miguel Angel Marchuet > > Luis Krause Mantilla escribió: > > Ron: > > > > It seems my email is choppy. I sent this msg but didn't make it to > the > > list: > > > > I found the answer. Miguel changed the previous default logic: > > > > 2009-01-28 16:46 UTC+0100 Miguel Angel Marchuet > <miguelan...@marchuet.net> > > * include\hbsetup.h > > + added HB_NO_TRACE as default, when trace will be required need > add > > macro HB_PRG_TRACE. > > By default this is turned off. Define HB_PRG_TRACE to turn it > on. > > > > No sure how much of an overhead this was, but having to rebuild > xharbour > > in order to > > trace can become a problem, especially if your building with stable > > version of xharbour. > > > > Regards, > > > > Ron Pinkas wrote: > > > >> Luis, > >> > >> AFAICT, no one touched anything related to this functionality in > years - I > >> suggest you carefully review again. > >> > >> Ron > >> > >> -------------------------------------------------- > >> From: "Luis Krause Mantilla" <lkrau...@shaw.ca> > >> Sent: Tuesday, March 10, 2009 11:13 AM > >> To: "Ron Pinkas" <ron.pin...@xharbour.com> > >> Cc: "Miguel Angel Marchuet" <miguelan...@marchuet.net>; > >> <modals...@yahoo.com.br>; "Xharbour-Developers List" > >> <xharbour-developers@lists.sourceforge.net> > >> Subject: Re: [xHarbour-developers] Upper() and codepage. > >> > >> > >> > >>> Ron > >>> > >>> > >>>> Anyone aware of any SHOW STOPPER, or would like to post any other > commit? > >>>> > >>>> > >>>> > >>> __TracePrgCalls( .t. ) has stopped working > >>> > >>> For us it has been helpful finding problems. > >>> > >>> Regards, > >>> > >>> -- > >>> Luis Krause Mantilla > >>> lkrausem at shaw dot ca > >>> luis_krause at hotmail dot com > >>> "May the Source be with GNU" > >>> > >>> > >>> > >>> > >>> > >> -------------------------------------------------------------------- > ---------- > >> _______________________________________________ > >> xHarbour-developers mailing list > >> xHarbour-developers@lists.sourceforge.net > >> https://lists.sourceforge.net/lists/listinfo/xharbour-developers > >> > >> > >> > > > > > > ----------------------------------------------------------------------- > ------- > Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are > powering Web 2.0 with engaging, cross-platform capabilities. Quickly > and > easily build your RIAs with Flex Builder, the Eclipse(TM)based > development > software that enables intelligent coding and step-through debugging. > Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com > _______________________________________________ > xHarbour-developers mailing list > xHarbour-developers@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/xharbour-developers > No virus found in this incoming message. > Checked by AVG - www.avg.com > Version: 8.0.237 / Virus Database: 270.11.8/1985 - Release Date: > 03/10/09 07:19:00 ------------------------------------------------------------------------------ Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com _______________________________________________ xHarbour-developers mailing list xHarbour-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/xharbour-developers
