Is it worth considering the case of `separator` being `nil`? This method will then blow up during string concatenation at the end. Since it wasn't taken into consideration originally, I wouldn't worry about it.