Hi April, Before you revert, pls lets check what is the proper syntax for PROCEDURE METHODs and change them to that.
If we find out, maybe we can do some optimizations in existing classes by converting RETURN NILs to PROC METHODs. Brgds, Viktor On Sat, Jun 20, 2009 at 9:31 PM, April White<april_wh...@sympatico.ca> wrote: > toni...@fwi wrote: >>>> >>>> * contrib\xhb\hblog.prg >>>> * PROCEDURE Open() CLASS HB_Logger -> METHOD returning Self >>>> * PROCEDURE Close() CLASS HB_Logger -> METHOD returning Self >>>> * PROCEDURE Log( cMessage, nPriority ) CLASS HB_Logger -> METHOD >>>> returning Self >>>> * PROCEDURE Log( nStyle, cMessage, cName, nPriority ) CLASS >>>> HB_LogChannel -> METHOD returning Self >>>> * PROCEDURE Send( nStyle, cMessage, cName, nPriority ) CLASS >>>> HB_LogConsole -> METHOD returning Self >>>> * PROCEDURE Send( nStyle, cMessage, cName, nPriority ) CLASS >>>> HB_LogDebug -> METHOD returning Self >>>> >>> >>> Is RETURN SELF correct? Shouldn't it be RETURN NIL? >>> >> >> +1... >> >> Some years ago: >> >> 2003-07-07 11:12 UTC-0300 Antonio Carlos Pantaglione >> * include/hbclass.ch >> + Added support for PROCEDURE <ProcName> CLASS <ClassName> syntax. >> >> I changed hbclass to support procedures for optimize pourpose. IMO >> only New() method need return Self. There are others exceptions like >> Redefine in FWH that need return Self too. >> > > Close() returning self was an oversight of mine. > > I'll revert my changes and ensure that the hbextern code excludes such > procedures. > > April > > -- > Some people are alive only because it is illegal to shoot them > > _______________________________________________ > Harbour mailing list > Harbour@harbour-project.org > http://lists.harbour-project.org/mailman/listinfo/harbour > _______________________________________________ Harbour mailing list Harbour@harbour-project.org http://lists.harbour-project.org/mailman/listinfo/harbour
