diff options
author | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-04-07 19:33:14 +0000 |
---|---|---|
committer | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-04-07 19:33:14 +0000 |
commit | 36d22d82aa202bb199967e9512281e9a53db42c9 (patch) | |
tree | 105e8c98ddea1c1e4784a60a5a6410fa416be2de /devtools/shared/sprintfjs/UPGRADING.md | |
parent | Initial commit. (diff) | |
download | firefox-esr-36d22d82aa202bb199967e9512281e9a53db42c9.tar.xz firefox-esr-36d22d82aa202bb199967e9512281e9a53db42c9.zip |
Adding upstream version 115.7.0esr.upstream/115.7.0esrupstream
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'devtools/shared/sprintfjs/UPGRADING.md')
-rw-r--r-- | devtools/shared/sprintfjs/UPGRADING.md | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/devtools/shared/sprintfjs/UPGRADING.md b/devtools/shared/sprintfjs/UPGRADING.md new file mode 100644 index 0000000000..e0db44df67 --- /dev/null +++ b/devtools/shared/sprintfjs/UPGRADING.md @@ -0,0 +1,17 @@ +SPRINTF JS UPGRADING + +Original library at https://github.com/alexei/sprintf.js + +This library should no longer be upgraded from upstream. We added performance improvements +in https://bugzilla.mozilla.org/show_bug.cgi?id=1406311. Most importantly removing the +usage of the get_type() method as well as prioritizing the %S use case. + +If for some reason, updating from upstream becomes necessary, please refer to the bug +mentioned above to reimplement the performance fixes in the new version. + +By default the library only supports string placeholders using %s (lowercase) while we use +%S (uppercase). The library also has to be manually patched in order to support it. + +- grab the unminified version at https://github.com/alexei/sprintf.js/blob/master/src/sprintf.js +- update the re.placeholder regexp to allow "S" as well as "s" +- update the switch statement in the format() method to make case "S" equivalent to case "s" |