win32utils devel - Aug 2006 - [ win32utils-Bugs-5503 ] Process::create() checks return value of CreateProcess against 0, should be false

Bugs item #5503, was opened at 2006-08-23 21:27
You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=411&aid=5503&group_id=85

Category: win32-process
Group: Code
Status: Open
Resolution: None
Priority: 3
Submitted By: David Haney (darius42)
Assigned to: Nobody (None)
Summary: Process::create() checks return value of CreateProcess against 0,
should be false

Initial Comment:
In Process::create(), the return value of CreateProcess() is compared against 0
to determine if the CreateProcess call succeeds.  It appears that instead of 0,
CreateProcess() will return false on a failure, and this comparison will fail
(no exception).  Instead a ProcessInfo with all 0 values will be returned.  If
the code is changed to compare against false, an exception is thrown on failure.

----------------------------------------------------------------------

You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=411&aid=5503&group_id=85

Bugs item #5503, was opened at 2006-08-23 21:27
You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=411&aid=5503&group_id=85

Category: win32-process
Group: Code
>Status: Closed
>Resolution: Accepted
Priority: 3
Submitted By: David Haney (darius42)
>Assigned to: Daniel Berger (djberg96)
Summary: Process::create() checks return value of CreateProcess against 0,
should be false

Initial Comment:
In Process::create(), the return value of CreateProcess() is compared against 0
to determine if the CreateProcess call succeeds.  It appears that instead of 0,
CreateProcess() will return false on a failure, and this comparison will fail
(no exception).  Instead a ProcessInfo with all 0 values will be returned.  If
the code is changed to compare against false, an exception is thrown on failure.

----------------------------------------------------------------------
>Comment By: Daniel Berger (djberg96)
Date: 2006-08-24 20:14

Message:
Good catch!  I''ve fixed it in CVS and I''ll have a new release
out shortly.

Thanks,

Dan

----------------------------------------------------------------------

You can respond by visiting: 
http://rubyforge.org/tracker/?func=detail&atid=411&aid=5503&group_id=85