I think the rearrangement of parameters is unnecessary and doesn't
would make sense, but any way it's not worth changing in my opinion.
I think that in most peoples code there would only be one line of
code constructing a Connection.
If you were changing something that people used all the time then
maybe it would be worth doing, but this is not.
Maybe a struct to contain all the connection parameters would be better.
Connection::Connection(const ConnectionSettings & cs)
This is the sort of thing I do in my code where it wraps