Rename StringBuilder::flexibleAppend(...) to StringBuilder::append(...)
Created attachment 376349 [details] Patch
Comment on attachment 376349 [details] Patch Attachment 376349 [details] did not pass jsc-ews (mac): Output: https://webkit-queues.webkit.org/results/12915434 New failing tests: mozilla-tests.yaml/js1_5/Array/regress-101964.js.mozilla-dfg-eager-no-cjit-validate-phases
Created attachment 376584 [details] Patch
Created attachment 376585 [details] Patch
Created attachment 376608 [details] Patch
Created attachment 376609 [details] Patch
Comment on attachment 376609 [details] Patch A next cleanup step would be to remove other overloads of StringBuilder::append that are redundant because append(...) does the same thing (even with one argument).
And then another is to change most appendNumber callers to just call append, since it does there right thing. And remove most appendNumber overloads. Then consider removing appendFixedPrecisionNumber and appendFixedWidthNumber and use FormattedNumber instead.
Comment on attachment 376609 [details] Patch Clearing flags on attachment: 376609 Committed r248822: <https://trac.webkit.org/changeset/248822>
All reviewed patches have been landed. Closing bug.
<rdar://problem/54430185>