DeprecatedString::format is broken on windows
DeprecatedString::setNum(99) for example fails to set the contents of the string to "99". What's going on is that the initial call to vsnprintf fails, and so the string is assigned an empty value.
There's a comment saying that the code from String::format should be duplicated for DeprecatedString::format! Is that still the plan?
This bug causes things like "window.location.href" to report an URL that is lacking the port number, which causes some pages not to work properly.
By the way, I think the correct solution is to use _vscprintf instead of calling vsnprintf in a loop like String::format does. Patch coming up...
Created attachment 11661 [details]
Use _vscprintf in both places.
By the way, the documentation for _vscprintf is here:
Comment on attachment 11661 [details]
Looks great, r=me.
Committed revision 17985.