Closed
Bug 108766
Opened 23 years ago
Closed 23 years ago
extend nsICmdLineHandler to expose a method that gets called on startup
Categories
(MailNews Core :: Simple MAPI, defect, P1)
Tracking
(Not tracked)
mozilla0.9.7
People
(Reporter: rdayal, Assigned: kkhandrika)
References
(Blocks 1 open bug)
Details
There is a need to execute some code as part of Mozilla startup when mozilla is
started with 'MAPIStartup' command line arguments. The current nsICmdLineHandler
is tied to bringing up a window using a chrome URL where as in this case Mozilla
should come up with no UI and hence no chrome is associated with it.
A default implementation of this method should be provided and could be
overwritten by different modules to do things specific to that module.
As per suggestion recieved from Seth and Mike Shaver, this method should look like :
PRUint32 onCommandLineSwitch(in [array,size_is(len)] string argv, in
PRUint32 len, in PRUint32 offset);
where offset is the argc at which the switch was found, and the return value is
the number of arguments consumed.
This method should be called from an appropriate place in nsAppRunner.
Quick, how does this differ from 56654's intent, which is to add the ability to
perform arbitrary processing at startup, based on the command-line interface?
There's even a patch there, and stuff.
Comment 2•23 years ago
|
||
I thought the idea was to turn bug 56654 into the rewrite from scratch bug, and
this one would track the hack over the current stuff.
I think we should do what we do for all the other API freezings, and file a
master nsICmdLineInterface freeze bug, with 56654 and other more specific
complaints as dependencies.
56654 has a (tested, but long ago) patch to do exactly what this bug is
purporting to want, with minimal change to the existing interface, which seems
basically _exactly_ what this bug needs.
I think that we should wait and fix 56654 in the context of a more complete
clean-up, but that holds true for this effective duplicate as well.
Comment 4•23 years ago
|
||
This bug needs to be fixed for simple MAPI trunk landing.
For detailed explanation, please see bug 102821. (Browser page started up when I
initiate a simple MAPI transaction.) mscott produced a patch for this bug for
simple MAPI branch landing at that time, and suggested that the patch
needs to be moved to a better place for trunk landing. Copied below comments
#10 in that bug.
------ Additional Comment #10 From mscott@netscape.com 2001-10-08 14:43 -------
btw, this patch would not go into the trunk. For this to go into the trunk the
hacked up logic I added to nsAppRunner.cpp would need to get moved to a new
class in the mapi code. This class would support command line handling for mapi
(nsICommandLineHandler)....
Priority: -- → P1
Target Milestone: --- → mozilla0.9.7
Comment 5•23 years ago
|
||
For more info about why this is needed for trunk landing of sMAPI, see also bug
103785.
Trunk landing for simple MAPI is the NO.1 objective for the simple MAPI project
team.
Comment 6•23 years ago
|
||
*** This bug has been marked as a duplicate of 56654 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•