Closed
Bug 1189619
Opened 10 years ago
Closed 10 years ago
Port bug Bug 1188745 [Rename nsTArray::SizeOfExcludingThis() as ShallowSizeOfExcludingThis() ] to comm-central
Categories
(MailNews Core :: Build Config, defect)
Tracking
(firefox42 fixed, seamonkey2.39 fixed)
RESOLVED
FIXED
Thunderbird 42.0
People
(Reporter: misak.bugzilla, Assigned: misak.bugzilla)
References
Details
Attachments
(1 file, 1 obsolete file)
3.11 KB,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
We should make changes for comm-central part of bug 1188745 in order to able to build.
Assignee | ||
Comment 1•10 years ago
|
||
This patch allows to build on my Fedora 22 x64
Attachment #8641526 -
Flags: review?(Pidgeot18)
![]() |
||
Updated•10 years ago
|
Component: MailNews: Backend → Build Config
Product: SeaMonkey → MailNews Core
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment on attachment 8641526 [details] [diff] [review]
Replace nsTArray SizeOfIncludingThis with ShallowSizeOfIncludingThis
Sorry, this looks wrong:
>+++ b/mailnews/db/msgdb/public/nsDBFolderInfo.h
> virtual size_t SizeOfExcludingThis(mozilla::MallocSizeOf aMallocSizeOf) const
> {
>- return m_lateredKeys.SizeOfExcludingThis(aMallocSizeOf);
>+ return m_lateredKeys.ShallowSizeOfIncludingThis(aMallocSizeOf);
Changing from Excluding to Including?!?
>+++ b/mailnews/db/msgdb/public/nsMsgHdr.h
> size_t SizeOfExcludingThis(mozilla::MallocSizeOf aMallocSizeOfFun) const
> {
>- return m_references.SizeOfExcludingThis(aMallocSizeOfFun);
>+ return m_references.ShallowSizeOfIncludingThis(aMallocSizeOfFun);
Changing from Excluding to Including?!?
>+++ b/mailnews/db/msgdb/src/nsMsgDatabase.cpp
>- totalSize += m_newSet.SizeOfExcludingThis(aMallocSizeOf);
>- totalSize += m_ChangeListeners.SizeOfExcludingThis(aMallocSizeOf);
>- totalSize += m_threads.SizeOfExcludingThis(aMallocSizeOf);
>+ totalSize += m_newSet.ShallowSizeOfIncludingThis(aMallocSizeOf);
>+ totalSize += m_ChangeListeners.ShallowSizeOfExcludingThis(aMallocSizeOf);
>+ totalSize += m_threads.ShallowSizeOfIncludingThis(aMallocSizeOf);
Changing from Excluding to Including?!?
Attachment #8641526 -
Flags: feedback-
Assignee | ||
Comment 9•10 years ago
|
||
Sorry, my first patch after 2 years :(
Attachment #8641526 -
Attachment is obsolete: true
Attachment #8641526 -
Flags: review?(Pidgeot18)
Attachment #8641639 -
Flags: review?(Pidgeot18)
Attachment #8641639 -
Flags: feedback?(iann_bugzilla)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•10 years ago
|
Attachment #8641639 -
Flags: review?(Pidgeot18)
Attachment #8641639 -
Flags: review+
Attachment #8641639 -
Flags: feedback?(iann_bugzilla)
![]() |
||
Comment 22•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-seamonkey2.39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 42.0
You need to log in
before you can comment on or make changes to this bug.
Description
•