SockAddr constructor should take sockaddr, not sockaddr_storage

The SockAddr constructor:
SockAddr::SockAddr(const sockaddr_storage& other, socklen_t size)
will deep copy sockaddr_storage based on sizeof(sockaddr_storage).

The sockaddr_storage type should only be used for storing data and not be a constructor parameter. The constructor should instead take a (sockaddr*, socklen_t).