[GitHub] [nuttx] pussuw commented on pull request #8486: sched/nxtask_sigchild: Set process exit code to group exit code

[via GitHub]

pussuw commented on PR #8486:
URL: https://github.com/apache/nuttx/pull/8486#issuecomment-1424903447

   I am a bit confused at this point, the test where the failure occurs is a 
bit odd.
   
   ```
     else if (WEXITSTATUS(stat_loc) != RETURN_STATUS)
       {
         printf("waitpid_test: ERROR: "
                "PID %d return status is %d, expected %d\n",
                g_waitpids[0], WEXITSTATUS(stat_loc), RETURN_STATUS);
         ASSERT(false);
       }
   ```
   Is this testing the child return code ? I'm very confused. The original test 
seems to be a bit incomplete, as it sets #define RETURN_STATUS 14 for every one 
and then tests if all return this.
   
   I'm not at all certain how to fix this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org