Closed Bug 887048 Opened 11 years ago Closed 11 years ago

Use mozilla::MallocSizeOf instead of nsMallocSizeOfFun

Categories

(MailNews Core :: Database, defect)

defect
Not set
critical

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 25.0

People

(Reporter: hiro, Assigned: hiro)

Details

Attachments

(1 file)

Bug 798914 introduced MallocSizeOf. We need to follow it to fix build errors:



                 from /home/zoe/hg/comm-central/mailnews/base/util/nsMsgDBFolder.cpp:18:
../../../mozilla/dist/include/nsMsgHdr.h:44:32: エラー: ‘nsMallocSizeOfFun’ has not been declared
../../../mozilla/dist/include/nsMsgHdr.h:48:32: エラー: ‘nsMallocSizeOfFun’ has not been declared
../../../mozilla/dist/include/nsMsgHdr.h: メンバ関数 ‘size_t nsMsgHdr::SizeOfExcludingThis(int) const’ 内:
../../../mozilla/dist/include/nsMsgHdr.h:46:63: エラー: ‘int’ から ‘size_t (*)(const void*) {aka long unsigned int (*)(const void*)}’ への無効な変換です [-fpermissive]
../../../mozilla/dist/include/nsTArray.h:833:10: エラー:   initializing argument 1 of ‘size_t nsTArray_Impl<E, Alloc>::SizeOfExcludingThis(mozilla::MallocSizeOf) const [with E = nsCString, Alloc = nsTArrayInfallibleAllocator, size_t = long unsigned int, mozilla::MallocSizeOf = long unsigned int (*)(const void*)]’ [-fpermissive]
../../../mozilla/dist/include/nsMsgHdr.h: メンバ関数 ‘size_t nsMsgHdr::SizeOfIncludingThis(int) const’ 内:
../../../mozilla/dist/include/nsMsgHdr.h:50:35: エラー: ‘aMallocSizeOfFun’ cannot be used as a function
In file included from ../../../mozilla/dist/include/nsMsgDatabase.h:18:0,

Sorry for the Japanese messages.
Attached patch FixSplinter Review
Attachment #767490 - Flags: review?(mbanner)
Comment on attachment 767490 [details] [diff] [review]
Fix

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

Looks great, thanks for fixing the bustage. I'll land it now.
Attachment #767490 - Flags: review?(mbanner) → review+
https://hg.mozilla.org/comm-central/rev/63f411eb52f3
Assignee: nobody → hiikezoe
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: