search

alphacomm / alpharpc

AlphaRPC is an RPC daemon based on ZeroMQ that can be used to distribute workload over one or multiple servers.
alpharpc.net
BSD 3-Clause "New" or "Revised" License
2 stars 5 forks source link

Refactor ClientHandler. #29

Closed jacobkiers closed 10 years ago

jacobkiers commented 10 years ago

Mainly readability improvements. There are no functional changes in this PR as far as I could understand.

Aside from commit 331c98d, all commits only touch a single method, which makes reviewing the changes easier. That was also done to make sure there are no functional changes.

jacobkiers commented 10 years ago

Still has to be tested. The test suite passes, though.

reenl commented 10 years ago

Nice job on the early returns and code style!

Important is that you check clientRequest comments before we merge.