Closed
Bug 1336958
Opened 7 years ago
Closed 7 years ago
Call delete on HTML::Tree objects to avoid leaking memory
Categories
(bugzilla.mozilla.org :: Extensions, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dylan, Assigned: dylan)
References
Details
Attachments
(1 file)
1.23 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
Reading the documentation of HTML::Tree https://metacpan.org/pod/HTML::Tree#SYNOPSIS Compare that with what we actually do in these two extensions: - https://github.com/mozilla-bteam/bmo/blob/master/extensions/SecureMail/Extension.pm#L645 - https://github.com/mozilla-bteam/bmo/blob/master/extensions/Profanivore/Extension.pm#L156
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8833963 -
Flags: review?(dkl)
Comment 2•7 years ago
|
||
Comment on attachment 8833963 [details] [diff] [review] 1336958_1.patch Review of attachment 8833963 [details] [diff] [review]: ----------------------------------------------------------------- r=dkl
Attachment #8833963 -
Flags: review?(dkl) → review+
Assignee | ||
Comment 3•7 years ago
|
||
To git@github.com:mozilla-bteam/bmo.git dc47938..92ca9f6 master -> master
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 4•7 years ago
|
||
This is now live. dkl
Updated•5 years ago
|
Component: Extensions: SecureMail → Extensions
You need to log in
before you can comment on or make changes to this bug.
Description
•