Skip to content

Don't leak a global variable#249

Open
etiennebarrie wants to merge 1 commit intorubychan:masterfrom
etiennebarrie:dont-leak-global-variable
Open

Don't leak a global variable#249
etiennebarrie wants to merge 1 commit intorubychan:masterfrom
etiennebarrie:dont-leak-global-variable

Conversation

@etiennebarrie
Copy link

While debugging in a pry console, I noticed there was a $DEBUG_BEFORE global variable and I tracked it down to here.

It's not necessary to use a global variable to store the previous value of $DEBUG. Since it's in the same scope, a local variable is enough.

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 8813276 and detected 0 issues on this pull request.

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant