This is just cleanup. Using a pointer instead of reference used to be necessary
before the Connection constructor took a make_server option. But now there is
no reason to have a pointer and unnecessary null handling.
Also, Connection add_client option and AddClient RemoveClient ServeStream
wrapper functions are removed. These were originally meant to simplify usages
but cause more trouble than they are worth.
This is just cleanup. Using a pointer instead of reference used to be necessary before the Connection constructor took a make_server option. But now there is no reason to have a pointer and unnecessary null handling.
Also, Connection add_client option and AddClient RemoveClient ServeStream wrapper functions are removed. These were originally meant to simplify usages but cause more trouble than they are worth.