gustavonihei commented on PR #6137: URL: https://github.com/apache/incubator-nuttx/pull/6137#issuecomment-1106743590
> Yeah, but POSIX does not define an error code if `start_loc` is `NULL`, so Linux interpreted that to an error code. For me it is natural that if `start_loc` is `NULL` then the caller "does not care" about exit status. I agree with your point. In summary: - Prior to this PR, `wait` and `waitpid` had a premise of `stat_loc` being non-NULL. Passing a NULL pointer resulted in undefined behavior when DEBUG assertions are not enabled. - This PR defines a proper behavior for a NULL `stat_loc`. I am okay with the proposed changes. -- 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
