Closed Bug 1089441 Opened 5 years ago Closed 5 years ago
.pl and bloatdiff .pl in the same directory
We have tools/memory/bloattable.pl and tools/rb/bloatdiff.pl which are both used to process bloat logs, and so should live together. One possibility is to move bloattable.pl into tools/rb/. Another is to create a new directory (tools/bloatview/, perhaps) and move them both in there. Which of these you prefer probably depends on whether you consider BloatView and the refcount balancer as part of a single thing, or distinct things; I lean towards the latter. dbaron, do you have a preference?
I don't think I have a strong opinion here, although I'd probably lean towards separate from tools/rb/. (Sorry for the delay; it's taken a while to catch up from being away.)
This is the new home for these docs.
Attachment #8521838 - Flags: review?(dbaron)
(In reply to Nicholas Nethercote [:njn] from comment #2) > Created attachment 8521837 [details] [diff] [review] > Move bloattable.pl and bloatdiff.pl into tools/bloatview/ Note that because this patch only renames files, the "Review" and "Diff" links don't show anything.
Attachment #8521837 - Flags: review?(dbaron) → review+
Attachment #8521838 - Flags: review?(dbaron) → review+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
I updated https://developer.mozilla.org/en-US/docs/Mozilla/Performance/BloatView to reflect the new locations.
You need to log in before you can comment on or make changes to this bug.