zorba-coders team mailing list archive

["Paul J. Lucas" <paul@xxxxxxxxxxxxx>]

Review: Needs Fixing

1. Why return made-up numbers, e.g., -1000, etc?  Why not simply return the actual exit status?  Of course this implies that you'd have to implement XQuery functions that do the equivalent of the W* macros.  Either that, or return XML data, e.g.:

  <process id="1234" raw-exit-code="0" status="normal"/>

or:

  <process id="1234" raw-exit-code="56" status="signal" signal="5" dumped-core="true"/>

2. When you throw the USER_EXCEPTION, why not include the actual OS-level error message?  There is code in src/util/error_util.h.  Yes, this implies that this code would have to be moved to the public API.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1086323-processExitCode/+merge/137966
Your team Zorba Coders is subscribed to branch lp:zorba/process-module.