← Back to branch summary

~gnomefreak/firefox-extensions/firegpg.ubuntu

« back to all changes in this revision

Viewing changes to FireGPGCall/ipc/response.txt

  • Committer: John Vivirito
  • Date: 2008-08-12 11:47:33 UTC
  • Revision ID: gnomefreak@ubuntu.com-20080812114733-hn73tjxi26ylibrf
* import of upstream source version 0.5.1

Show diffs side-by-side

added added

removed removed

Lines of Context:
FireGPGCall/ipc/response.txt
 
1
 
 
2
> The api that I think should move to nsIProcess is GetEnv.  This interface,
 
3
> already has API related to the running of an exe, namely Kill().
 
4
 
 
5
I'll remove GetEnv from the IPC extension as soon as you add it to nsIProcess
 
6
 
 
7
> Should the Open() parameters on the nsIPipeConsole be turned into preferences -
 
8
> or at least #define's?
 
9
 
 
10
IPC does not have a UI at this time
 
11
 
 
12
> In ExecPipe, you init most of the out vars, but not all of them to null.  You
 
13
> should also init env?
 
14
 
 
15
env is actually an input variable! (an array of strings)
 
16
 
 
17
> nsPipeChannel
 
18
> This class is not threadsafe.  Not sure if you intend it to be.
 
19
 
 
20
Do nsIChannel implementations need to be thread-safe? If not, I'd leave it
 
21
the way it is.