search

CrowdHailer / Ace

HTTP web server and client, supports http1 and http2
https://hex.pm/packages/ace
MIT License
305 stars 26 forks source link

Seems this can be simplified further #78

Closed CharlesOkwuagwu closed 6 years ago

CharlesOkwuagwu commented 6 years ago

https://github.com/CrowdHailer/Ace/blob/58d19db2eff6ca80248e97e2e863058619f1c8e5/lib/ace/http/worker.ex#L88-L100

Regardless of the value of response.body, you return {[response], state}

CrowdHailer commented 6 years ago

Yes would agree. It obviously works without those commented parts