Closed Bug 920695 Opened 8 years ago Closed 8 years ago

Link xpcshell functionality into libxul, make xpcshell.exe a little stub

Categories

(Core :: XPConnect, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: benjamin, Assigned: benjamin)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

In order to stop exporting JS symbols, we need to have a solution for xpcshell, and the simplest solution is just to build xpcshell into libxul and export it. Mine!
Blocks: 920731
Comment on attachment 810127 [details] [diff] [review]
part A - move xpcshell code into libxul and make the xpcshell binary just a stub, r?bholley

Review of attachment 810127 [details] [diff] [review]:
-----------------------------------------------------------------

I didn't read through the cut-paste here. If there are any significant changes in there that you want me to review, please upload some kind of reduced diff.

r=bholley with comments addressed.

::: js/xpconnect/shell/xpcshell.cpp
@@ +30,3 @@
>  {
>  #ifdef XP_MACOSX
> +  InitAutoreleasePool();

XPConnect uses 4-space indentation. I don't care a huge about for this stub file, but we should probably be consistent.

::: js/xpconnect/src/moz.build
@@ +49,5 @@
>      'nsCxPusher.cpp',
>      'nsScriptError.cpp',
>      'nsXPConnect.cpp',
>      'Sandbox.cpp',
> +    'shellimpl.cpp',

Let's call this XPCShellImpl.cpp.
Attachment #810127 - Flags: review?(bobbyholley+bmo) → review+
Comment on attachment 810130 [details] [diff] [review]
part B - Remove references to JS_READLINE/EDITLINE since its in a different place now and won't work, r?bholley

Review of attachment 810130 [details] [diff] [review]:
-----------------------------------------------------------------

Hopefully nobody's using this. r=holley
Attachment #810130 - Flags: review?(bobbyholley+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/9e6846fd74a2
https://hg.mozilla.org/mozilla-central/rev/e403b2561ceb
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.