Seems weird storing the packet data in an instance var when you're really only using it in `send_packet`. Why not just make it an argument to that method?