Closed Bug 220406 Opened 21 years ago Closed 21 years ago

nsLocalFileWin::Launch should return more specific errors

Categories

(Core :: XPCOM, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.6alpha

People

(Reporter: Biesinger, Assigned: Biesinger)

Details

Attachments

(1 file)

msdn describes the specific error returns of ShellExecute
Attachment #132220 - Flags: superreview?(darin)
Attachment #132220 - Flags: review?(dougt)
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.6alpha
Comment on attachment 132220 [details] [diff] [review]
patch

looks fine.  thanks for the patch.  r=dougt
Attachment #132220 - Flags: review?(dougt) → review+
uhm... so how do you plan to make use of these error codes?  what bug does this
block?  if you don't plan to use these error codes, then why add this code?
your not talking about alot of code.  (14 branches and some overhead code).
Comment on attachment 132220 [details] [diff] [review]
patch

true, but it is still a valid question, right? ;-)
Attachment #132220 - Flags: superreview?(darin) → superreview+
I'm planning to use it in the ExternalHelperAppService code, to show users more
friendly error messages when launching a helper application failed. bug is not
yet filed, I will cc you.
Checking in nsLocalFileWin.cpp;
/cvsroot/mozilla/xpcom/io/nsLocalFileWin.cpp,v  <--  nsLocalFileWin.cpp
new revision: 1.109; previous revision: 1.108
done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: