Closed
Bug 270107
Opened 21 years ago
Closed 21 years ago
nsProcess::Run is broken
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
Details
Attachments
(1 file)
|
1.55 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
Symptoms:
/Users/timeless/mozilla/xpcom/threads/nsProcessCommon.cpp:350: warning:
assignment of negative value `PR_FAILURE' to `nsresult'
/Users/timeless/mozilla/xpcom/threads/nsProcessCommon.cpp:350: warning: argument
of negative value `PR_FAILURE' to `unsigned int'
Attachment #166071 -
Flags: review?(dougt)
|
||
Comment 2•21 years ago
|
||
Comment on attachment 166071 [details] [diff] [review]
match types
looks fine. Please rename "rv" to something like status. you never return
"rv", so the variable name is incorrect.
Attachment #166071 -
Flags: review?(dougt) → review+
mozilla/xpcom/threads/nsProcessCommon.cpp 1.16
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Comment 4•20 years ago
|
||
*** Bug 238115 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•