Closed cristaloleg closed 3 weeks ago
sendMessage was doing a bit too much, observability could happen on the caller side, which easier to adapt and extend in the future.
sendMessage
Also, dropping 1 if in processResponses 'cause it's always false. And slice prealloc, looks harmless.
if
processResponses
false
Overview
sendMessagewas doing a bit too much, observability could happen on the caller side, which easier to adapt and extend in the future.Also, dropping 1
ifinprocessResponses'cause it's alwaysfalse. And slice prealloc, looks harmless.