Last Comment Bug 750290 - Remove tools/leaky
: Remove tools/leaky
Status: RESOLVED FIXED
: dev-doc-needed
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla15
Assigned To: Mike Hommey [:glandium]
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-04-30 08:43 PDT by Mike Hommey [:glandium]
Modified: 2014-10-14 01:47 PDT (History)
4 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch (99.61 KB, patch)
2012-04-30 08:43 PDT, Mike Hommey [:glandium]
dbaron: review+
Details | Diff | Splinter Review

Description Mike Hommey [:glandium] 2012-04-30 08:43:40 PDT
Created attachment 619579 [details] [diff] [review]
Patch

Leaky isn't built by default, isn't built on any tinderbox build and doesn't support anything other than x86 (not even x86-64). I think it can safely go away.
Comment 3 Nicholas Nethercote [:njn] 2014-05-19 23:04:56 PDT
XPCOM_MEM_LEAKY_LOG still exists, and there's some code in xpcom/base/nsTraceRefcnt.cpp that relies on it -- search for |gLogToLeaky|. Can that code be removed?
Comment 4 David Baron :dbaron: ⌚️UTC-7 (busy September 14-25) 2014-05-19 23:11:56 PDT
I believe it can.
Comment 5 Mike Hommey [:glandium] 2014-05-19 23:21:00 PDT
Cf. dbaron
Comment 6 Nicholas Nethercote [:njn] 2014-10-14 01:47:34 PDT
> XPCOM_MEM_LEAKY_LOG still exists, and there's some code in
> xpcom/base/nsTraceRefcnt.cpp that relies on it -- search for |gLogToLeaky|.

It's gone now.

Note You need to log in before you can comment on or make changes to this bug.