I don't think we have discussed about infinite loop in our style guide. This is my personal opinion so let me know what you think.

I would rather use `while (sending)` and replace the two break statements with `sending = false`. Generally speaking I would avoid while(true) or similar pattern. It's more readable this way; no need to read the whole loop body to understand what's going one with the branching and breaking.