Closed
Bug 825901
Opened 12 years ago
Closed 12 years ago
Remove the old linker code
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Future
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files, 1 obsolete file)
150.88 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
22.31 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
mobile/xul builds for android are pretty much dead, so I think it's safe to remove the old linker code, now. This will make mobile/xul use the new linker, which would lead to library memory not being shared between processes anymore, but it's not like it really matters, since we don't produce such builds. And memory sharing between processes is under work for the new linker, for webapps, anyways, so it'd be solved in a few weeks/months.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #697014 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 2•12 years ago
|
||
Forgot a part in APKOpen.cpp
Attachment #697034 -
Flags: review?(blassey.bugs)
Assignee | ||
Updated•12 years ago
|
Attachment #697014 -
Attachment is obsolete: true
Attachment #697014 -
Flags: review?(blassey.bugs)
Updated•12 years ago
|
Attachment #697034 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2deb0d26f529
Assignee | ||
Comment 4•12 years ago
|
||
I forgot to remove the headers, too.
Attachment #697406 -
Flags: review?(blassey.bugs)
Assignee | ||
Updated•12 years ago
|
Whiteboard: [leave open]
Updated•12 years ago
|
Attachment #697406 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b28771946018
Whiteboard: [leave open]
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/2deb0d26f529 https://hg.mozilla.org/mozilla-central/rev/b28771946018
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Future
You need to log in
before you can comment on or make changes to this bug.
Description
•