<div dir="ltr"><div>This is close but has a few minor things.</div><div><br></div><div>Initialize infop = {0} gets rid of compilation errors on Linux.</div><div>waitid() is not prototyped by RTEMS. File a POSIX-Compliance ticket for this and put it in the section of methods tracked by tickets.</div><div>Add a blank line before the return.</div><div>No need to initailization a variable which is passed by address (&var).</div><div><br></div><div>Please tidy up and resubmit.<br></div><div><br></div><div>--joel<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 12, 2018 at 3:14 AM Abhimanyu Raghuvanshi <<a href="mailto:abhimanyuraghuvanshi29@gmail.com">abhimanyuraghuvanshi29@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Patch attached<br><br>======================<br><div>From 54e11c76b2e112c3b0e33838d0057b335d6d4e43 Mon Sep 17 00:00:00 2001</div><div>From: ABR290B <<a href="mailto:abhimanyuraghuvanshi29@gmail.com" target="_blank">abhimanyuraghuvanshi29@gmail.com</a>></div><div>Date: Wed, 12 Dec 2018 14:42:53 +0530</div><div>Subject: [PATCH] POSIX Signature Test for wait.h (GCI2018)</div><div><br></div><div>---</div><div> testsuites/psxtests/Makefile.am | 5 ++-</div><div> testsuites/psxtests/psxhdrs/wait/wait.c | 35 ++++++++++++++++++++</div><div> testsuites/psxtests/psxhdrs/wait/waitid.c | 38 ++++++++++++++++++++++</div><div> testsuites/psxtests/psxhdrs/wait/waitpid.c | 38 ++++++++++++++++++++++</div><div> 4 files changed, 115 insertions(+), 1 deletion(-)</div><div> create mode 100644 testsuites/psxtests/psxhdrs/wait/wait.c</div><div> create mode 100644 testsuites/psxtests/psxhdrs/wait/waitid.c</div><div> create mode 100644 testsuites/psxtests/psxhdrs/wait/waitpid.c</div><div><br></div><div>diff --git a/testsuites/psxtests/Makefile.am b/testsuites/psxtests/Makefile.am</div><div>index 3dd8fe0139..1ad3078261 100644</div><div>--- a/testsuites/psxtests/Makefile.am</div><div>+++ b/testsuites/psxtests/Makefile.am</div><div>@@ -1815,7 +1815,10 @@ lib_a_SOURCES = psxhdrs/devctl/posix_devctl.c \</div><div> <span style="white-space:pre-wrap"> </span>psxhdrs/setjmp/longjmp.c \</div><div> <span style="white-space:pre-wrap"> </span>psxhdrs/setjmp/setjmp.c \</div><div> <span style="white-space:pre-wrap"> </span>psxhdrs/setjmp/siglongjmp.c \</div><div>-<span style="white-space:pre-wrap"> </span>psxhdrs/setjmp/sigsetjmp.c</div><div>+<span style="white-space:pre-wrap"> </span>psxhdrs/setjmp/sigsetjmp.c \</div><div>+<span style="white-space:pre-wrap"> </span>psxhdrs/wait/wait.c \</div><div>+<span style="white-space:pre-wrap"> </span>psxhdrs/wait/waitid.c \</div><div>+<span style="white-space:pre-wrap"> </span>psxhdrs/wait/waitpid.c </div><div> </div><div> ## Not supported by RTEMS, but POSIX API Compliance tests exist.</div><div> ## lib_a_SOURCES += psxhdrs/ulimit/ulimit.c</div><div>diff --git a/testsuites/psxtests/psxhdrs/wait/wait.c b/testsuites/psxtests/psxhdrs/wait/wait.c</div><div>new file mode 100644</div><div>index 0000000000..30408e86d3</div><div>--- /dev/null</div><div>+++ b/testsuites/psxtests/psxhdrs/wait/wait.c</div><div>@@ -0,0 +1,35 @@</div><div>+/**</div><div>+ * @file</div><div>+ * @brief wait() API Conformance Test</div><div>+ */</div><div>+</div><div>+/*</div><div>+ * COPYRIGHT (c) 2018.</div><div>+ * Abhimanyu Raghuvanshi</div><div>+ *</div><div>+ * Permission to use, copy, modify, and/or distribute this software</div><div>+ * for any purpose with or without fee is hereby granted.</div><div>+ * </div><div>+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL</div><div>+ * WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED</div><div>+ * WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR</div><div>+ * BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES</div><div>+ * OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS,</div><div>+ * WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION,</div><div>+ * ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.</div><div>+ */</div><div>+ </div><div>+#ifdef HAVE_CONFIG_H</div><div>+#include "config.h"</div><div>+#endif</div><div>+</div><div>+#include <sys/types.h></div><div>+#include <sys/wait.h></div><div>+</div><div>+int test(void);</div><div>+</div><div>+int test(void)</div><div>+{</div><div>+ int pid = 1234;</div><div>+ return wait(&pid) != -1;</div><div>+}</div><div>\ No newline at end of file</div><div>diff --git a/testsuites/psxtests/psxhdrs/wait/waitid.c b/testsuites/psxtests/psxhdrs/wait/waitid.c</div><div>new file mode 100644</div><div>index 0000000000..8f20f24466</div><div>--- /dev/null</div><div>+++ b/testsuites/psxtests/psxhdrs/wait/waitid.c</div><div>@@ -0,0 +1,38 @@</div><div>+/**</div><div>+ * @file</div><div>+ * @brief waitid() API Conformance Test</div><div>+ */</div><div>+</div><div>+/*</div><div>+ * COPYRIGHT (c) 2018.</div><div>+ * Abhimanyu Raghuvanshi</div><div>+ *</div><div>+ * Permission to use, copy, modify, and/or distribute this software</div><div>+ * for any purpose with or without fee is hereby granted.</div><div>+ * </div><div>+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL</div><div>+ * WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED</div><div>+ * WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR</div><div>+ * BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES</div><div>+ * OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS,</div><div>+ * WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION,</div><div>+ * ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.</div><div>+ */</div><div>+ </div><div>+#ifdef HAVE_CONFIG_H</div><div>+#include "config.h"</div><div>+#endif</div><div>+</div><div>+#include <sys/types.h></div><div>+#include <sys/wait.h></div><div>+</div><div>+int test(void);</div><div>+</div><div>+int test(void)</div><div>+{</div><div>+ idtype_t P_PID = 1;</div><div>+ id_t id = 0;</div><div>+ siginfo_t infop = {0,0,0};</div><div>+ int pid = 1234;</div><div>+ return waitid(P_PID, id, &infop, pid) != -1;</div><div>+}</div><div>\ No newline at end of file</div><div>diff --git a/testsuites/psxtests/psxhdrs/wait/waitpid.c b/testsuites/psxtests/psxhdrs/wait/waitpid.c</div><div>new file mode 100644</div><div>index 0000000000..4a95f8528e</div><div>--- /dev/null</div><div>+++ b/testsuites/psxtests/psxhdrs/wait/waitpid.c</div><div>@@ -0,0 +1,38 @@</div><div>+/**</div><div>+ * @file</div><div>+ * @brief waitpid() API Conformance Test</div><div>+ */</div><div>+</div><div>+/*</div><div>+ * COPYRIGHT (c) 2018.</div><div>+ * Abhimanyu Raghuvanshi</div><div>+ *</div><div>+ * Permission to use, copy, modify, and/or distribute this software</div><div>+ * for any purpose with or without fee is hereby granted.</div><div>+ * </div><div>+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL</div><div>+ * WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED</div><div>+ * WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR</div><div>+ * BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES</div><div>+ * OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS,</div><div>+ * WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION,</div><div>+ * ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.</div><div>+ */</div><div>+ </div><div>+#ifdef HAVE_CONFIG_H</div><div>+#include "config.h"</div><div>+#endif</div><div>+</div><div>+#include <sys/types.h></div><div>+#include <sys/wait.h></div><div>+</div><div>+int test(void);</div><div>+</div><div>+int test(void)</div><div>+{</div><div>+ pid_t pid = 2;</div><div>+ int WEXITSTATUS = 1;</div><div>+ int wcontinued = 0;</div><div>+ </div><div>+ return waitpid(pid, &WEXITSTATUS, wcontinued) != -1;</div><div>+}</div><div>\ No newline at end of file</div><div>-- </div><div>2.19.1.windows.1</div><div>=====================================</div><div><br></div><div><div dir="ltr" class="gmail-m_-5793392812318277669gmail_signature"><div dir="ltr"><div><br></div><div><font face="arial black, sans-serif">ABR</font></div></div></div></div></div></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>