Gnomish Yellow Pages

22 - names leaking into global scope

Using r58.

The OnLeave, OnEnter, OnClick handlers set up for TradeButton:Create around line 700 are showing up in global scope.  They're in a do..end block, but that does not implicitly make them local.  (Declaring them local makes them local *to that block* as opposed to the file, but without a local declaration it doesn't matter how far down they're nested, they're still global by default.)

Ditto for "Config", whatever that is.  :-)

User When Change
lilsparky Jan 12, 2011 at 22:41 UTC Changed status from New to Accepted
Farmbuyer Jan 12, 2011 at 22:28 UTC Create

You must login to post a comment. Don't have an account? Register to get one!

  • 2 comments
  • Avatar of Farmbuyer Farmbuyer Jan 21, 2011 at 23:33 UTC - 0 likes

    Totally know how that goes. I want to go back and erase my very first addon from historical existence, it's so embarrassing....

  • Avatar of lilsparky lilsparky Jan 12, 2011 at 22:41 UTC - 0 likes

    indeed. you can tell gyp was still a learning process for me with "Config" being the name of my savedvar. :(

  • 2 comments

Facts

Last updated
Jan 12, 2011
Reported
Jan 12, 2011
Status
Accepted - Problem reproduced / Need acknowledged.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees