Good afternoon Jafar, Thank you. I myself am making a number of stylistic changes to my own code for readability purposes. This is another one I think I will include.
regards Bruce Rennie On 25/09/14 14:23, Jafar Al-Gharaibeh wrote: > Bruce, > > The change is purely for style just to make it consistent with most > references in the gui class library. > using or not using self is entirely a programmer preference, it has no > effect on the end result unless there is a name collision with a local > variable, in that case self is used to reference the class variable as > apposed to a local variable with the same name. > > Personally I like to use self when I reference class variables as a > way to improve readability. self makes it obvious that I'm referencing > a class variable, not local, not global. But again, I don't always do > that, after all, x := y + z is a a lot shorter and cleaner than self.x > := self.y + self.z! > > Cheers, > Jafar > > > On Wed, Sep 24, 2014 at 10:52 PM, Bruce & Breeanna Rennie > <[email protected] <mailto:[email protected]>> wrote: > > Jafar has made the following change to teh code for component.icn > > */cwin* to */self.cwin* > > Is this a matter of style or is there a side issue that I am not aware > of? As I thought that it is okay to not have the self as part of the > reference. > > regards > > Bruce Rennie > > > ------------------------------------------------------------------------------ > Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer > Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS > Reports > Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper > Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer > > http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk > _______________________________________________ > Unicon-group mailing list > [email protected] > <mailto:[email protected]> > https://lists.sourceforge.net/lists/listinfo/unicon-group > > > > > ------------------------------------------------------------------------------ > Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer > Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports > Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper > Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer > http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk > > > _______________________________________________ > Unicon-group mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/unicon-group ------------------------------------------------------------------------------ Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk _______________________________________________ Unicon-group mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/unicon-group
