Closed
Bug 407445
Opened 17 years ago
Closed 2 years ago
make-tree.pl treats "--object" address case-sensitively
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: jruderman, Unassigned)
Details
Attachments
(1 file, 1 obsolete file)
7.96 KB,
patch
|
dbaron
:
review-
|
Details | Diff | Splinter Review |
I ran into this when trying to debug bug 407277. The output from trace-refcnt included the address of the leaked object in lowercase: 1476 @0x3f13ef90 (1 references; 0 from COMPtrs) but make-tree.pl gave me nothing for "--object 0x3f13ef90". It worked fine when I retried with "--object 0x3F13EF90". I only figured that out because find-leakers.pl outputs the address in uppercase. IMO, make-tree.pl should work given an address in either case.
This just requires changing this line: next LINE unless ($obj eq $::opt_object); to do a case-insensitive comparison. (Probably either call lc on both arguments -- maybe on $::opt_object once outside the loop -- or use the i parameter to a regexp match.)
Reporter | ||
Comment 2•17 years ago
|
||
I don't know perl, so can you do it? :)
Updated•17 years ago
|
QA Contact: general → xpcom
Comment 3•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008080121 Minefield/3.1a2pre] (home, debug default) (W2Ksp4) Uppercase the parameter value. Unrelated: *Move a line to optimize a loop a little. *Add/remove a few blank lines.
Updated•16 years ago
|
Attachment #332291 -
Flags: review? → review?(dbaron)
Updated•16 years ago
|
Severity: normal → minor
Component: XPCOM → Testing
OS: Mac OS X → All
QA Contact: xpcom → testing
Hardware: PC → All
Target Milestone: --- → mozilla1.9.1a2
Comment on attachment 332291 [details] [diff] [review] (Av1) <make-tree.pl> 1) don't reformat all the code 2) don't assume that the stuff in the log is one case or the other. It may vary between platforms. (However, find-leakers and make-tree are consistent.)
Attachment #332291 -
Flags: review?(dbaron) → review-
Comment 5•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1a2pre) Gecko/2008080121 Minefield/3.1a2pre] (home, debug default) (W2Ksp4) Av1, with comment 4 suggestion(s), plus: *Merge |$callGraphRoot| declar + init lines. *Add a few comments. *Remove unused |my ...|s. *Remove unreachable code. *Add forward declarations, to silence warnings. *And add a "missing" parameter value. *Indent one line.
Attachment #332308 -
Flags: review?(dbaron)
Updated•16 years ago
|
Attachment #332291 -
Attachment is obsolete: true
Comment on attachment 332308 [details] [diff] [review] (Av2) <make-tree.pl> Don't reformat all the code and make lots of arbitrary changes, especially given that this has no test suite.
Attachment #332308 -
Flags: review?(dbaron) → review-
Comment 7•16 years ago
|
||
There's probably not a good component for bugs like these.
Component: Testing → General
Product: Core → Testing
QA Contact: testing → general
Target Milestone: mozilla1.9.1a2 → ---
Comment 8•2 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months.
:ahal, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee: bugzillamozillaorg_serge_20140323 → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(ahal)
Updated•2 years ago
|
Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(ahal)
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•