<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 3:27 PM Vijay Kumar Banerjee <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@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"><br><br><br><div class="gmail_quote"><div dir="ltr">On Thu, 6 Dec 2018 at 02:03, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</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>Vijay</div><div><br></div><div>Do you still see it with the committed version of the patch? a couple of methods are listed in the section where file names are comments out with tickets beside them. <br></div><div><br></div></div></blockquote><div>I still see the warning in the committed version if I compile getline.c with -Wall  </div></div></div></blockquote><div><br></div><div>If you compile by hand you will but it is disabled in the Makefile.am with this comment: <br></div><div><br></div><div>## Specific issues that tickets are tracking<br>## lib_a_SOURCES += psxhdrs/dirent/dirfd.c       See ticket #3371<br>## lib_a_SOURCES += psxhdrs/dirent/fdopendir.c   See ticket #3371<br>## lib_a_SOURCES += psxhdrs/stdio/getdelim.c     See ticket #3633<br>## lib_a_SOURCES += psxhdrs/stdio/getline.c      See ticket #3633<br></div><div><br></div><div>I think this is OK until we address the issue in newlib.</div><div><br></div><div>And this is precisely the type of issue these tests are intended to find. :)<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 class="gmail_quote"><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></div><div>--joel<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 2:26 PM Vijay Kumar Banerjee <<a href="mailto:vijaykumar9597@gmail.com" target="_blank">vijaykumar9597@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"><div dir="ltr"><br><br><br><div class="gmail_quote"><div dir="ltr">On Thu, 6 Dec 2018 at 01:39, Himanshu Sekhar Nayak <<a href="mailto:himanshuwindows8.1@gmail.com" target="_blank">himanshuwindows8.1@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>Hi guys,</div><div><br></div><div>Here is the updated patch.</div><div><br></div></div></blockquote><div>There's warning in getline.c even after defining _POSIX_C_SOURCE </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></div><div>Thanks</div><div>Himanshu<br></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></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>
</blockquote></div></div>
</blockquote></div></div></div>