1353580 - Remove support for !HAVE_PR_MEMMAP in mozJSComponentLoader.cpp

Status: RESOLVED FIXED

(Core :: XPConnect, enhancement)

Description

[Andrew McCreight [:mccr8]]

There's a decent chunk of code to support the !HAVE_PR_MEMMAP case in mozJSComponentLoader.cpp, but it is unconditionally defined. It looks like it used to be undefined for BEOS and OS2.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1353580 - Remove support for !HAVE_PR_MEMMAP in mozJSComponentLoader.

This was only ever used for BeOS and OS2, which have likely long ago
stopped working for other reasons.

Review commit: https://reviewboard.mozilla.org/r/126856/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/126856/

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8854935 [details]
Bug 1353580 - Remove support for !HAVE_PR_MEMMAP in mozJSComponentLoader.

https://reviewboard.mozilla.org/r/126856/#review129562

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0cc784ece282
Remove support for !HAVE_PR_MEMMAP in mozJSComponentLoader. r=mrbkap

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/0cc784ece282