public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
* Missing include patch
@ 2017-08-21  8:50 Dirk Vanden Boer
  2017-08-21 12:20 ` Fritz Reese
  0 siblings, 1 reply; 2+ messages in thread
From: Dirk Vanden Boer @ 2017-08-21  8:50 UTC (permalink / raw)
  To: fortran

[-- Attachment #1: Type: text/plain, Size: 148 bytes --]

Hi,

I submitted an issue a while back about a missing include: bug id 79134.
They told me patches need to be sent here.

Regards,
Dirk Vanden Boer

[-- Attachment #2: missing-include.patch --]
[-- Type: application/octet-stream, Size: 311 bytes --]

--- gcc-6.3.0/libgfortran/io/close.c.org	2017-01-17 09:43:48.395850000 +0100
+++ gcc-6.3.0/libgfortran/io/close.c	2017-01-17 09:21:05.000000000 +0100
@@ -25,6 +25,7 @@
 #include "io.h"
 #include "unix.h"
 #include <limits.h>
+#include <stdlib.h>
 
 typedef enum
 { CLOSE_DELETE, CLOSE_KEEP, CLOSE_UNSPECIFIED }

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Missing include patch
  2017-08-21  8:50 Missing include patch Dirk Vanden Boer
@ 2017-08-21 12:20 ` Fritz Reese
  0 siblings, 0 replies; 2+ messages in thread
From: Fritz Reese @ 2017-08-21 12:20 UTC (permalink / raw)
  To: Dirk Vanden Boer; +Cc: fortran

On Mon, Aug 21, 2017 at 4:50 AM, Dirk Vanden Boer <dirk.vdb@gmail.com> wrote:
> Hi,
>
> I submitted an issue a while back about a missing include: bug id 79134.
> They told me patches need to be sent here.
>
> Regards,
> Dirk Vanden Boer

This is fixed by r243846 (committed last year as part of gcc-7.1.0)
which moves many separate includes of stdlib.h to libgfortran.h .

The patch looks simple enough but we should consider whether it's
worth backporting r243846 or committing this small patch (to gcc-6).

---
Fritz Reese

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-08-21 12:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-21  8:50 Missing include patch Dirk Vanden Boer
2017-08-21 12:20 ` Fritz Reese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).