Don't assume swap returns void