work on SwapValue Todos (mostly length parameter in constructor)