Backed out changeset e396dfbd2f76 (bug 1905712)
[gecko.git] / devtools / shared / sprintfjs / UPGRADING.md
blobe0db44df675933f128811f9009796780ff1f60d9
1 SPRINTF JS UPGRADING
3 Original library at https://github.com/alexei/sprintf.js
5 This library should no longer be upgraded from upstream. We added performance improvements
6 in https://bugzilla.mozilla.org/show_bug.cgi?id=1406311. Most importantly removing the
7 usage of the get_type() method as well as prioritizing the %S use case.
9 If for some reason, updating from upstream becomes necessary, please refer to the bug
10 mentioned above to reimplement the performance fixes in the new version.
12 By default the library only supports string placeholders using %s (lowercase) while we use
13 %S (uppercase). The library also has to be manually patched in order to support it.
15 - grab the unminified version at https://github.com/alexei/sprintf.js/blob/master/src/sprintf.js
16 - update the re.placeholder regexp to allow "S" as well as "s"
17 - update the switch statement in the format() method to make case "S" equivalent to case "s"