Closed
Bug 572863
Opened 14 years ago
Closed 14 years ago
performance runtests needs code refactor
Categories
(Tamarin Graveyard :: Tools, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cpeyer, Assigned: cpeyer)
References
Details
Attachments
(1 file, 2 obsolete files)
52.46 KB,
patch
|
dschaffe
:
review+
|
Details | Diff | Splinter Review |
The runtest function w/in performance/runtests.py needs to be cleaned up and modularized.
Assignee | ||
Comment 1•14 years ago
|
||
only new functionality added is enabling of debug output for performance runtests passes. Otherwise this is all code refactor and cleanup.
Assignee | ||
Comment 2•14 years ago
|
||
Full patch with lots of whitespace fixes too.
Attachment #452108 -
Attachment is obsolete: true
Attachment #452306 -
Flags: review?(dschaffe)
Attachment #452108 -
Flags: review?(dschaffe)
Assignee | ||
Comment 3•14 years ago
|
||
Memory runs were not working after initial patch - this fixes those issues.
Attachment #452306 -
Attachment is obsolete: true
Attachment #452316 -
Flags: review?(dschaffe)
Attachment #452306 -
Flags: review?(dschaffe)
Updated•14 years ago
|
Attachment #452316 -
Flags: review?(dschaffe) → review+
Comment 4•14 years ago
|
||
Comment on attachment 452316 [details] [diff] [review] fix issues with memory runs in previous patch reviewed changes and ran ok on windows
Assignee | ||
Comment 5•14 years ago
|
||
Pushed to redux: http://hg.mozilla.org/tamarin-redux/rev/a8278f593389
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•