New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove some unused variables from main #419

Closed

slashnick wants to merge 1 commit into memcached:master from slashnick:master

Contributor

slashnick commented Aug 15, 2018

Since l_socket and u_socket are both checked and freed, the compiler didn't notice that they are otherwise unused.


          Remove some unused variables from main

466db87

dormando added the merged/fixed for next label

Member

dormando commented Aug 15, 2018

nice catch. looks like they've been dangling since 2008! it's merged for the next release.

dormando closed this

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