search

allegro / marathon-consul

Integrates Marathon apps with Consul service discovery.
Apache License 2.0
191 stars 33 forks source link

Move defer out of function body. #259

Closed janisz closed 7 years ago

janisz commented 7 years ago

After sse.NewHandler was moved to fucntion, defer was called everytime handler was created not just before exiting main.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 90.066% when pulling bf3bf49a222a597c742f8cabde38231b82ba3ba0 on hotfix/defer_called_every_time into e108886e17a2a26f143ca9b946a07c91ac300f0f on master.