names leaking into global scope #22

  • Defect
  • Accepted
Open
Assigned to
View _ForgeUser280725's Profile
_ForgeUser280725
  • View Farmbuyer's Profile
    Farmbuyer created this issue Jan 12, 2011

    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.  :-)

  • View Farmbuyer's Profile
    Farmbuyer     added the tags New Defect Jan 12, 2011
  • View _ForgeUser280725's Profile
    _ForgeUser280725 posted a comment Jan 12, 2011

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

  • View _ForgeUser280725's Profile
    _ForgeUser280725     removed a tag New Jan 12, 2011
  • View _ForgeUser280725's Profile
    _ForgeUser280725     added a tag Accepted Jan 12, 2011
  • View Farmbuyer's Profile
    Farmbuyer posted a comment Jan 21, 2011

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

To post a comment, please login or register a new account.