unify nsLegacyCheck, nsRunApp (and nsObjectIgnore)

RESOLVED INVALID

Status

SeaMonkey
Installer
RESOLVED INVALID
14 years ago
10 years ago

People

(Reporter: Andrew Schultz, Assigned: Andrew Schultz)

Tracking

({memory-footprint})

Trunk
All
Linux
memory-footprint

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

35.94 KB, patch
Benjamin Smedberg
: review+
Details | Diff | Splinter Review
(Assignee)

Description

14 years ago
the legacy check, run app and object-to-ignore classes are all identical (a list
of strings).  legacy check and run app have two strings per object, while
object-to-ignore has only one string, but since it's allocated dynamically, it
should be fine.

Updated

14 years ago
QA Contact: bugzilla → agracebush
Product: Browser → Seamonkey
(Assignee)

Comment 1

14 years ago
Created attachment 174815 [details] [diff] [review]
patch

I took the easy way out
nsLegacyCheck, nsObjectIgnore, nsRunApp => nsStringList
(Assignee)

Updated

14 years ago
Attachment #174815 - Flags: review?(benjamin)

Updated

13 years ago
Attachment #174815 - Flags: review?(benjamin) → review+
(Assignee)

Updated

13 years ago
Attachment #174815 - Flags: superreview?(dveditz)
There is no Linux installer on Sm-trunk anymore. Does this bug still have any validity?
QA Contact: agracebush → general
(Assignee)

Comment 3

10 years ago
this can die
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → INVALID
Attachment #174815 - Flags: superreview?(dveditz)
You need to log in before you can comment on or make changes to this bug.