Bug 18047 – std.format value.length modulo seperator step width leads to wrong length

@safe unittest { auto cmp = " 123,456"; assert(cmp.length == 12, format("%d", cmp.length)); auto tmp = format("%12,d", 123456); assert(tmp.length == 12, format("%d", tmp.length)); assert(tmp == cmp, "'" ~ tmp ~ "'"); } The length of tmp should be 12, but it is 11. Does not depend on number type. The error also occurs with floats.

Commit pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/aa92422b215c7109bd74bb1cad1226654b7b6f3b Merge pull request #5911 from burner/issue_17459_issue18047 Fix for two problems with format separators merged-on-behalf-of: Andrei Alexandrescu <[email protected]>

Commit pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/aa92422b215c7109bd74bb1cad1226654b7b6f3b Merge pull request #5911 from burner/issue_17459_issue18047