Closed Bug 9204 Opened 27 years ago Closed 26 years ago

apprunner main() needs cleaning up

Categories

(SeaMonkey :: UI Design, defect, P1)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: sfraser_bugs, Assigned: davidm)

Details

One of the things that mcmullen was going to do was to clean up apprunner main(), which is currently 580 lines of crap long. Becuase startup is notoriously fragile because of ordering issues, main() should be really clean, and specify precicely the order in which things happen. Most of the code currently in main() should be moved out to other functions, and main() should be no longer than a few lines long.
Assignee: don → law
OS: Mac System 8.5 → All
Priority: P3 → P1
Target Milestone: M10
Bill, do you have the bandwidth to fix this in M10?
I talked to David Matiskella about this last week. I think he is working on it. I've added him to the cc: list. David, if you think this falls under the category of what you were working on, please reassign this to yourself.
Assignee: law → davidm
Target Milestone: M10 → M9
I'll do an initial pass. Componentization of things like command handling should be a seperate bug. Going to try to do for m9 as the longer I wait the more crap is going to get shoved into main.
Status: NEW → ASSIGNED
Target Milestone: M9 → M10
m10
Target Milestone: M10 → M14
Moving off to m14. I have done a bit of cleanup but some more needs to be done after things like AppCores no longer exist and when the command line handlers get moved off into to seperate modules.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I have decided to mark this fixed and open a new bug about having to cleanup cmd line handling
Status: RESOLVED → VERIFIED
This has been mostly done. I filed a bug on morse about the cookie and wallet stuff in here, but it's better.
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.