Removing unused variable and whitespace

brg-liuwei / ngx_kafka_module

nginx kafka module, send post log data to kafka cluster

Other

176 stars 64 forks source link

Removing unused variable and whitespace #4

Closed davidalber closed 8 years ago

davidalber commented 8 years ago

Hi there, @brg-liuwei. Thanks for this NGINX module code! I have been looking through it to understand it and made a few changes in the process. I'm opening a few pull requests in case you are interested in merging them back into your repository, but if you prefer not to do that, no worries!

This pull request removes the declaration of an unused variable and a couple instances of trailing whitespace.

brg-liuwei commented 8 years ago

Thank you very much, PR#4 and PR#5 is accepted. I will review PR#6 for more time. I am very glad that you contribute those useful code to the project, especially PR#4 (remove the unused variables, which means you reviewed very carefully).

And, ..., I hope you can understand my pool English :)