]> git.ipfire.org Git - thirdparty/gcc.git/blobdiff - gcc/fortran/io.c
Update copyright years.
[thirdparty/gcc.git] / gcc / fortran / io.c
index cff2b689786726dd50845ac944c7ef814e5c7b43..da6ad177ec39e0681d601e9505eb720754c58a50 100644 (file)
@@ -1,5 +1,5 @@
 /* Deal with I/O statements & related stuff.
-   Copyright (C) 2000-2015 Free Software Foundation, Inc.
+   Copyright (C) 2000-2021 Free Software Foundation, Inc.
    Contributed by Andy Vaught
 
 This file is part of GCC.
@@ -25,10 +25,11 @@ along with GCC; see the file COPYING3.  If not see
 #include "gfortran.h"
 #include "match.h"
 #include "parse.h"
+#include "constructor.h"
 
 gfc_st_label
 format_asterisk = {0, NULL, NULL, -1, ST_LABEL_FORMAT, ST_LABEL_FORMAT, NULL,
-                  0, {NULL, NULL}};
+                  0, {NULL, NULL}, NULL};
 
 typedef struct
 {
@@ -38,6 +39,15 @@ typedef struct
 io_tag;
 
 static const io_tag
+       tag_readonly    = {"READONLY", " readonly", NULL, BT_UNKNOWN },
+       tag_shared      = {"SHARE", " shared", NULL, BT_UNKNOWN },
+       tag_noshared    = {"SHARE", " noshared", NULL, BT_UNKNOWN },
+       tag_e_share     = {"SHARE", " share =", " %e", BT_CHARACTER },
+       tag_v_share     = {"SHARE", " share =", " %v", BT_CHARACTER },
+       tag_cc          = {"CARRIAGECONTROL", " carriagecontrol =", " %e",
+                          BT_CHARACTER },
+       tag_v_cc        = {"CARRIAGECONTROL", " carriagecontrol =", " %v",
+                          BT_CHARACTER },
        tag_file        = {"FILE", " file =", " %e", BT_CHARACTER },
        tag_status      = {"STATUS", " status =", " %e", BT_CHARACTER},
        tag_e_access    = {"ACCESS", " access =", " %e", BT_CHARACTER},
@@ -102,7 +112,6 @@ static gfc_dt *current_dt;
 
 #define RESOLVE_TAG(x, y) if (!resolve_tag (x, y)) return false;
 
-
 /**************** Fortran 95 FORMAT parser  *****************/
 
 /* FORMAT tokens returned by format_lex().  */
@@ -113,7 +122,7 @@ enum format_token
   FMT_RPAREN, FMT_X, FMT_SIGN, FMT_BLANK, FMT_CHAR, FMT_P, FMT_IBOZ, FMT_F,
   FMT_E, FMT_EN, FMT_ES, FMT_G, FMT_L, FMT_A, FMT_D, FMT_H, FMT_END,
   FMT_ERROR, FMT_DC, FMT_DP, FMT_T, FMT_TR, FMT_TL, FMT_STAR, FMT_RC,
-  FMT_RD, FMT_RN, FMT_RP, FMT_RU, FMT_RZ
+  FMT_RD, FMT_RN, FMT_RP, FMT_RU, FMT_RZ, FMT_DT
 };
 
 /* Local variables for checking format strings.  The saved_token is
@@ -191,23 +200,14 @@ unget_char (void)
 /* Eat up the spaces and return a character.  */
 
 static char
-next_char_not_space (bool *error)
+next_char_not_space ()
 {
   char c;
   do
     {
       error_element = c = next_char (NONSTRING);
       if (c == '\t')
-       {
-         if (gfc_option.allow_std & GFC_STD_GNU)
-           gfc_warning (0, "Extension: Tab character in format at %C");
-         else
-           {
-             gfc_error ("Extension: Tab character in format at %C");
-             *error = true;
-             return c;
-           }
-       }
+       gfc_warning (OPT_Wtabs, "Nonconforming tab character in format at %C");
     }
   while (gfc_is_whitespace (c));
   return c;
@@ -225,7 +225,6 @@ format_lex (void)
   char c, delim;
   int zflag;
   int negative_flag;
-  bool error = false;
 
   if (saved_token != FMT_NONE)
     {
@@ -234,7 +233,7 @@ format_lex (void)
       return token;
     }
 
-  c = next_char_not_space (&error);
+  c = next_char_not_space ();
   
   negative_flag = 0;
   switch (c)
@@ -244,7 +243,7 @@ format_lex (void)
       /* Falls through.  */
 
     case '+':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       if (!ISDIGIT (c))
        {
          token = FMT_UNKNOWN;
@@ -255,7 +254,7 @@ format_lex (void)
 
       do
        {
-         c = next_char_not_space (&error);
+         c = next_char_not_space ();
          if (ISDIGIT (c))
            value = 10 * value + c - '0';
        }
@@ -285,7 +284,7 @@ format_lex (void)
 
       do
        {
-         c = next_char_not_space (&error);
+         c = next_char_not_space ();
          if (ISDIGIT (c))
            {
              value = 10 * value + c - '0';
@@ -320,7 +319,7 @@ format_lex (void)
       break;
 
     case 'T':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       switch (c)
        {
        case 'L':
@@ -348,7 +347,7 @@ format_lex (void)
       break;
 
     case 'S':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       if (c != 'P' && c != 'S')
        unget_char ();
 
@@ -356,7 +355,7 @@ format_lex (void)
       break;
 
     case 'B':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       if (c == 'N' || c == 'Z')
        token = FMT_BLANK;
       else
@@ -418,7 +417,7 @@ format_lex (void)
       break;
 
     case 'E':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       if (c == 'N' )
        token = FMT_EN;
       else if (c == 'S')
@@ -448,7 +447,7 @@ format_lex (void)
       break;
 
     case 'D':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       if (c == 'P')
        {
          if (!gfc_notify_std (GFC_STD_F2003, "DP format "
@@ -463,6 +462,55 @@ format_lex (void)
            return FMT_ERROR;
          token = FMT_DC;
        }
+      else if (c == 'T')
+       {
+         if (!gfc_notify_std (GFC_STD_F2003, "Fortran 2003: DT format "
+             "specifier not allowed at %C"))
+           return FMT_ERROR;
+         token = FMT_DT;
+         c = next_char_not_space ();
+         if (c == '\'' || c == '"')
+           {
+             delim = c;
+             value = 0;
+
+             for (;;)
+               {
+                 c = next_char (INSTRING_WARN);
+                 if (c == '\0')
+                   {
+                     token = FMT_END;
+                     break;
+                   }
+
+                 if (c == delim)
+                   {
+                     c = next_char (NONSTRING);
+                     if (c == '\0')
+                       {
+                         token = FMT_END;
+                         break;
+                       }
+                     if (c == '/')
+                       {
+                         token = FMT_SLASH;
+                         break;
+                       }
+                     if (c == delim)
+                       continue;
+                     unget_char ();
+                     break;
+                   }
+               }
+           }
+         else if (c == '/')
+           {
+             token = FMT_SLASH;
+             break;
+           }
+         else
+           unget_char ();
+       }
       else
        {
          token = FMT_D;
@@ -471,7 +519,7 @@ format_lex (void)
       break;
 
     case 'R':
-      c = next_char_not_space (&error);
+      c = next_char_not_space ();
       switch (c)
        {
        case 'C':
@@ -512,9 +560,6 @@ format_lex (void)
       break;
     }
 
-  if (error)
-    return FMT_ERROR;
-
   return token;
 }
 
@@ -547,14 +592,18 @@ token_to_string (format_token t)
 static bool
 check_format (bool is_input)
 {
-  const char *posint_required    = _("Positive width required");
-  const char *nonneg_required    = _("Nonnegative width required");
-  const char *unexpected_element  = _("Unexpected element %<%c%> in format "
-                                     "string at %L");
-  const char *unexpected_end     = _("Unexpected end of format string");
-  const char *zero_width         = _("Zero width in format descriptor");
-
-  const char *error;
+  const char *posint_required
+    = G_("Positive width required in format string at %L");
+  const char *nonneg_required
+    = G_("Nonnegative width required in format string at %L");
+  const char *unexpected_element 
+    = G_("Unexpected element %qc in format string at %L");
+  const char *unexpected_end
+    = G_("Unexpected end of format string in format string at %L");
+  const char *zero_width
+    = G_("Zero width in format descriptor in format string at %L");
+
+  const char *error = NULL;
   format_token t, u;
   int level;
   int repeat;
@@ -572,7 +621,7 @@ check_format (bool is_input)
     goto fail;
   if (t != FMT_LPAREN)
     {
-      error = _("Missing leading left parenthesis");
+      error = G_("Missing leading left parenthesis in format string at %L");
       goto syntax;
     }
 
@@ -601,7 +650,8 @@ format_item_1:
          level++;
          goto format_item;
        }
-      error = _("Left parenthesis required after %<*%>");
+      error = G_("Left parenthesis required after %<*%> in format string "
+                "at %L");
       goto syntax;
 
     case FMT_POSINT:
@@ -632,7 +682,7 @@ format_item_1:
        goto fail;
       if (t != FMT_P)
        {
-         error = _("Expected P edit descriptor");
+         error = G_("Expected P edit descriptor in format string at %L");
          goto syntax;
        }
 
@@ -640,7 +690,8 @@ format_item_1:
 
     case FMT_P:
       /* P requires a prior number.  */
-      error = _("P descriptor requires leading scale factor");
+      error = G_("P descriptor requires leading scale factor in format "
+                "string at %L");
       goto syntax;
 
     case FMT_X:
@@ -700,12 +751,22 @@ format_item_1:
     case FMT_A:
     case FMT_D:
     case FMT_H:
+    case FMT_DT:
       goto data_desc;
 
     case FMT_END:
       error = unexpected_end;
       goto syntax;
 
+    case FMT_RPAREN:
+      if (flag_dec_blank_format_item)
+       goto finished;
+      else
+       {
+         error = G_("Missing item in format string at %L");
+         goto syntax;
+       }
+
     default:
       error = unexpected_element;
       goto syntax;
@@ -733,7 +794,8 @@ data_desc:
          && t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES
          && t != FMT_D && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
        {
-         error = _("Comma required after P descriptor");
+         error = G_("Comma required after P descriptor in format string "
+                    "at %L");
          goto syntax;
        }
       if (t != FMT_COMMA)
@@ -744,10 +806,11 @@ data_desc:
              if (t == FMT_ERROR)
                goto fail;
            }
-          if (t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES && t != FMT_D
-             && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
+         if (t != FMT_F && t != FMT_E && t != FMT_EN && t != FMT_ES
+             && t != FMT_D && t != FMT_G && t != FMT_RPAREN && t != FMT_SLASH)
            {
-             error = _("Comma required after P descriptor");
+             error = G_("Comma required after P descriptor in format string "
+                        "at %L");
              goto syntax;
            }
        }
@@ -761,7 +824,8 @@ data_desc:
       t = format_lex ();
       if (t != FMT_POSINT)
        {
-         error = _("Positive width required with T descriptor");
+         error = G_("Positive width required with T descriptor in format "
+                    "string at %L");
          goto syntax;
        }
       break;
@@ -772,27 +836,31 @@ data_desc:
        goto fail;
       if (t == FMT_POSINT)
        break;
-
-      switch (gfc_notification_std (GFC_STD_GNU))
+      if (mode != MODE_FORMAT)
+       format_locus.nextc += format_string_pos;
+      if (t == FMT_ZERO)
        {
-         case WARNING:
-           if (mode != MODE_FORMAT)
-             format_locus.nextc += format_string_pos;
-           gfc_warning (0, "Extension: Missing positive width after L "
-                        "descriptor at %L", &format_locus);
-           saved_token = t;
-           break;
-
-         case ERROR:
-           error = posint_required;
-           goto syntax;
-
-         case SILENT:
-           saved_token = t;
-           break;
-
-         default:
-           gcc_unreachable ();
+         switch (gfc_notification_std (GFC_STD_GNU))
+           {
+             case WARNING:
+               gfc_warning (0, "Extension: Zero width after L "
+                            "descriptor at %L", &format_locus);
+               break;
+             case ERROR:
+               gfc_error ("Extension: Zero width after L "
+                            "descriptor at %L", &format_locus);
+               goto fail;
+             case SILENT:
+               break;
+             default:
+               gcc_unreachable ();
+           }
+       }
+      else
+       {
+         saved_token = t;
+         gfc_notify_std (GFC_STD_GNU, "Missing positive width after "
+                         "L descriptor at %L", &format_locus);
        }
       break;
 
@@ -840,7 +908,8 @@ data_desc:
          u = format_lex ();
          if (u == FMT_E)
            {
-             error = _("E specifier not allowed with g0 descriptor");
+             error = G_("E specifier not allowed with g0 descriptor in "
+                        "format string at %L");
              goto syntax;
            }
          saved_token = u;
@@ -849,12 +918,38 @@ data_desc:
 
       if (u != FMT_POSINT)
        {
+         if (flag_dec)
+           {
+             if (flag_dec_format_defaults)
+               {
+                 /* Assume a default width based on the variable size.  */
+                 saved_token = u;
+                 break;
+               }
+             else
+               {
+                 gfc_error ("Positive width required in format "
+                            "specifier %s at %L", token_to_string (t),
+                            &format_locus);
+                 saved_token = u;
+                 goto fail;
+               }
+           }
+
          format_locus.nextc += format_string_pos;
-         gfc_error ("Positive width required in format "
-                        "specifier %s at %L", token_to_string (t),
-                        &format_locus);
-         saved_token = u;
-         goto fail;
+         if (!gfc_notify_std (GFC_STD_F2018,
+                              "positive width required at %L",
+                              &format_locus))
+           {
+             saved_token = u;
+             goto fail;
+           }
+         if (flag_dec_format_defaults)
+           {
+             /* Assume a default width based on the variable size.  */
+             saved_token = u;
+             break;
+           }
        }
 
       u = format_lex ();
@@ -900,9 +995,7 @@ data_desc:
       if (u == FMT_ERROR)
        goto fail;
       if (u != FMT_E)
-       {
-         saved_token = u;
-       }
+       saved_token = u;
       else
        {
          u = format_lex ();
@@ -910,19 +1003,91 @@ data_desc:
            goto fail;
          if (u != FMT_POSINT)
            {
-             error = _("Positive exponent width required");
-             goto syntax;
+             if (u == FMT_ZERO)
+               {
+                 if (!gfc_notify_std (GFC_STD_F2018,
+                                     "Positive exponent width required in "
+                                     "format string at %L", &format_locus))
+                   {
+                     saved_token = u;
+                     goto fail;
+                   }
+               }
+             else
+               {
+                 error = G_("Positive exponent width required in format "
+                            "string at %L");
+                 goto syntax;
+               }
            }
        }
 
       break;
 
+    case FMT_DT:
+      t = format_lex ();
+      if (t == FMT_ERROR)
+       goto fail;
+      switch (t)
+       {
+       case FMT_RPAREN:
+         level--;
+         if (level < 0)
+           goto finished;
+         goto between_desc;
+
+       case FMT_COMMA:
+         goto format_item;
+
+       case FMT_COLON:
+         goto format_item_1;
+
+       case FMT_LPAREN:
+
+  dtio_vlist:
+         t = format_lex ();
+         if (t == FMT_ERROR)
+           goto fail;
+
+         if (t != FMT_POSINT)
+           {
+             error = posint_required;
+             goto syntax;
+           }
+
+         t = format_lex ();
+         if (t == FMT_ERROR)
+           goto fail;
+
+         if (t == FMT_COMMA)
+           goto dtio_vlist;
+         if (t != FMT_RPAREN)
+           {
+             error = G_("Right parenthesis expected at %C in format string "
+                        "at %L");
+             goto syntax;
+           }
+         goto between_desc;
+
+       default:
+         error = unexpected_element;
+         goto syntax;
+       }
+      break;
+
     case FMT_F:
       t = format_lex ();
       if (t == FMT_ERROR)
        goto fail;
       if (t != FMT_ZERO && t != FMT_POSINT)
        {
+         if (flag_dec_format_defaults)
+           {
+             /* Assume the default width is expected here and continue lexing.  */
+             value = 0; /* It doesn't matter what we set the value to here.  */
+             saved_token = t;
+             break;
+           }
          error = nonneg_required;
          goto syntax;
        }
@@ -940,7 +1105,8 @@ data_desc:
          /* Warn if -std=legacy, otherwise error.  */
          if (gfc_option.warn_std != 0)
            {
-             error = _("Period required in format specifier");
+             error = G_("Period required in format specifier in format "
+                        "string at %L");
              goto syntax;
            }
          if (mode != MODE_FORMAT)
@@ -992,8 +1158,17 @@ data_desc:
        goto fail;
       if (t != FMT_ZERO && t != FMT_POSINT)
        {
-         error = nonneg_required;
-         goto syntax;
+         if (flag_dec_format_defaults)
+           {
+             /* Assume the default width is expected here and continue lexing.  */
+             value = 0; /* It doesn't matter what we set the value to here.  */
+             saved_token = t;
+           }
+         else
+           {
+             error = nonneg_required;
+             goto syntax;
+           }
        }
       else if (is_input && t == FMT_ZERO)
        {
@@ -1005,9 +1180,7 @@ data_desc:
       if (t == FMT_ERROR)
        goto fail;
       if (t != FMT_PERIOD)
-       {
-         saved_token = t;
-       }
+       saved_token = t;
       else
        {
          t = format_lex ();
@@ -1135,7 +1308,7 @@ syntax:
   if (error == unexpected_element)
     gfc_error (error, error_element, &format_locus);
   else
-    gfc_error ("%s in format string at %L", error, &format_locus);
+    gfc_error (error, &format_locus);
 fail:
   rv = false;
 
@@ -1199,6 +1372,15 @@ gfc_match_format (void)
       return MATCH_ERROR;
     }
 
+  /* Before parsing the rest of a FORMAT statement, check F2008:c1206.  */
+  if ((gfc_current_state () == COMP_FUNCTION
+       || gfc_current_state () == COMP_SUBROUTINE)
+      && gfc_state_stack->previous->state == COMP_INTERFACE)
+    {
+      gfc_error ("FORMAT statement at %C cannot appear within an INTERFACE");
+      return MATCH_ERROR;
+    }
+
   if (gfc_statement_label == NULL)
     {
       gfc_error ("Missing format label at %C");
@@ -1241,36 +1423,6 @@ gfc_match_format (void)
 }
 
 
-/* Check for a CHARACTER variable.  The check for scalar is done in
-   resolve_tag.  */
-
-static bool
-check_char_variable (gfc_expr *e)
-{
-  if (e->expr_type != EXPR_VARIABLE || e->ts.type != BT_CHARACTER)
-    {
-      gfc_error("IOMSG must be a scalar-default-char-variable at %L", &e->where);
-      return false;
-    }
-  return true;
-}
-
-
-static bool
-is_char_type (const char *name, gfc_expr *e)
-{
-  gfc_resolve_expr (e);
-
-  if (e->ts.type != BT_CHARACTER)
-    {
-      gfc_error ("%s requires a scalar-default-char-expr at %L",
-                  name, &e->where);
-      return false;
-    }
-  return true;
-}
-
-
 /* Match an expression I/O tag of some sort.  */
 
 static match
@@ -1328,24 +1480,29 @@ match_vtag (const io_tag *tag, gfc_expr **v)
       return MATCH_ERROR;
     }
 
-  if (result->symtree->n.sym->attr.intent == INTENT_IN)
+  if (result->symtree)
     {
-      gfc_error ("Variable %s cannot be INTENT(IN) at %C", tag->name);
-      gfc_free_expr (result);
-      return MATCH_ERROR;
-    }
+      bool impure;
 
-  bool impure = gfc_impure_variable (result->symtree->n.sym);
-  if (impure && gfc_pure (NULL))
-    {
-      gfc_error ("Variable %s cannot be assigned in PURE procedure at %C",
-                tag->name);
-      gfc_free_expr (result);
-      return MATCH_ERROR;
-    }
+      if (result->symtree->n.sym->attr.intent == INTENT_IN)
+       {
+         gfc_error ("Variable %s cannot be INTENT(IN) at %C", tag->name);
+         gfc_free_expr (result);
+         return MATCH_ERROR;
+       }
+
+      impure = gfc_impure_variable (result->symtree->n.sym);
+      if (impure && gfc_pure (NULL))
+       {
+         gfc_error ("Variable %s cannot be assigned in PURE procedure at %C",
+                    tag->name);
+         gfc_free_expr (result);
+         return MATCH_ERROR;
+       }
 
-  if (impure)
-    gfc_unset_implicit_pure (NULL);
+      if (impure)
+       gfc_unset_implicit_pure (NULL);
+    }
 
   *v = result;
   return MATCH_YES;
@@ -1361,7 +1518,16 @@ match_out_tag (const io_tag *tag, gfc_expr **result)
 
   m = match_vtag (tag, result);
   if (m == MATCH_YES)
-    gfc_check_do_variable ((*result)->symtree);
+    {
+      if ((*result)->symtree)
+       gfc_check_do_variable ((*result)->symtree);
+
+      if ((*result)->expr_type == EXPR_CONSTANT)
+       {
+         gfc_error ("Expecting a variable at %L", &(*result)->where);
+         return MATCH_ERROR;
+       }
+    }
 
   return m;
 }
@@ -1400,10 +1566,101 @@ match_ltag (const io_tag *tag, gfc_st_label ** label)
 }
 
 
+/* Match a tag using match_etag, but only if -fdec is enabled.  */
+static match
+match_dec_etag (const io_tag *tag, gfc_expr **e)
+{
+  match m = match_etag (tag, e);
+  if (flag_dec && m != MATCH_NO)
+    return m;
+  else if (m != MATCH_NO)
+    {
+      gfc_error ("%s at %C is a DEC extension, enable with "
+                "%<-fdec%>", tag->name);
+      return MATCH_ERROR;
+    }
+  return m;
+}
+
+
+/* Match a tag using match_vtag, but only if -fdec is enabled.  */
+static match
+match_dec_vtag (const io_tag *tag, gfc_expr **e)
+{
+  match m = match_vtag(tag, e);
+  if (flag_dec && m != MATCH_NO)
+    return m;
+  else if (m != MATCH_NO)
+    {
+      gfc_error ("%s at %C is a DEC extension, enable with "
+                "%<-fdec%>", tag->name);
+      return MATCH_ERROR;
+    }
+  return m;
+}
+
+
+/* Match a DEC I/O flag tag - a tag with no expression such as READONLY.  */
+
+static match
+match_dec_ftag (const io_tag *tag, gfc_open *o)
+{
+  match m;
+
+  m = gfc_match (tag->spec);
+  if (m != MATCH_YES)
+    return m;
+
+  if (!flag_dec)
+    {
+      gfc_error ("%s at %C is a DEC extension, enable with "
+                "%<-fdec%>", tag->name);
+      return MATCH_ERROR;
+    }
+
+  /* Just set the READONLY flag, which we use at runtime to avoid delete on
+     close.  */
+  if (tag == &tag_readonly)
+    {
+      o->readonly |= 1;
+      return MATCH_YES;
+    }
+
+  /* Interpret SHARED as SHARE='DENYNONE' (read lock).  */
+  else if (tag == &tag_shared)
+    {
+      if (o->share != NULL)
+       {
+         gfc_error ("Duplicate %s specification at %C", tag->name);
+         return MATCH_ERROR;
+       }
+      o->share = gfc_get_character_expr (gfc_default_character_kind,
+         &gfc_current_locus, "denynone", 8);
+      return MATCH_YES;
+    }
+
+  /* Interpret NOSHARED as SHARE='DENYRW' (exclusive lock).  */
+  else if (tag == &tag_noshared)
+    {
+      if (o->share != NULL)
+       {
+         gfc_error ("Duplicate %s specification at %C", tag->name);
+         return MATCH_ERROR;
+       }
+      o->share = gfc_get_character_expr (gfc_default_character_kind,
+         &gfc_current_locus, "denyrw", 6);
+      return MATCH_YES;
+    }
+
+  /* We handle all DEC tags above.  */
+  gcc_unreachable ();
+}
+
+
 /* Resolution of the FORMAT tag, to be called from resolve_tag.  */
 
 static bool
-resolve_tag_format (const gfc_expr *e)
+resolve_tag_format (gfc_expr *e)
 {
   if (e->expr_type == EXPR_CONSTANT
       && (e->ts.type != BT_CHARACTER
@@ -1414,6 +1671,54 @@ resolve_tag_format (const gfc_expr *e)
       return false;
     }
 
+  /* Concatenate a constant character array into a single character
+     expression.  */
+
+  if ((e->expr_type == EXPR_ARRAY || e->rank > 0)
+      && e->ts.type == BT_CHARACTER
+      && gfc_is_constant_expr (e))
+    {
+      if (e->expr_type == EXPR_VARIABLE
+         && e->symtree->n.sym->attr.flavor == FL_PARAMETER)
+       gfc_simplify_expr (e, 1);
+
+      if (e->expr_type == EXPR_ARRAY)
+       {
+         gfc_constructor *c;
+         gfc_charlen_t n, len;
+         gfc_expr *r;
+         gfc_char_t *dest, *src;
+
+         if (e->value.constructor == NULL)
+          {
+            gfc_error ("FORMAT tag at %L cannot be a zero-sized array",
+                       &e->where);
+            return false;
+          }
+
+         n = 0;
+         c = gfc_constructor_first (e->value.constructor);
+         len = c->expr->value.character.length;
+         
+         for ( ; c; c = gfc_constructor_next (c))
+           n += len;
+
+         r = gfc_get_character_expr (e->ts.kind, &e->where, NULL, n);
+         dest = r->value.character.string;
+
+         for (c = gfc_constructor_first (e->value.constructor);
+            c; c = gfc_constructor_next (c))
+           {
+             src = c->expr->value.character.string;
+             for (gfc_charlen_t i = 0 ; i < len; i++)
+               *dest++ = *src++;
+           }
+
+         gfc_replace_expr (e, r);
+         return true;
+       }
+    }
+
   /* If e's rank is zero and e is not an element of an array, it should be
      of integer or character type.  The integer variable should be
      ASSIGNED.  */
@@ -1581,16 +1886,12 @@ match_open_element (gfc_open *open)
   match m;
 
   m = match_etag (&tag_e_async, &open->asynchronous);
-  if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", open->asynchronous))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_unit, &open->unit);
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_iomsg, &open->iomsg);
-  if (m == MATCH_YES && !check_char_variable (open->iomsg))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
   m = match_out_tag (&tag_iostat, &open->iostat);
@@ -1648,6 +1949,23 @@ match_open_element (gfc_open *open)
   if (m != MATCH_NO)
     return m;
 
+  /* The following are extensions enabled with -fdec.  */
+  m = match_dec_etag (&tag_e_share, &open->share);
+  if (m != MATCH_NO)
+    return m;
+  m = match_dec_etag (&tag_cc, &open->cc);
+  if (m != MATCH_NO)
+    return m;
+  m = match_dec_ftag (&tag_readonly, open);
+  if (m != MATCH_NO)
+    return m;
+  m = match_dec_ftag (&tag_shared, open);
+  if (m != MATCH_NO)
+    return m;
+  m = match_dec_ftag (&tag_noshared, open);
+  if (m != MATCH_NO)
+    return m;
+
   return MATCH_NO;
 }
 
@@ -1680,16 +1998,28 @@ gfc_free_open (gfc_open *open)
   gfc_free_expr (open->convert);
   gfc_free_expr (open->asynchronous);
   gfc_free_expr (open->newunit);
+  gfc_free_expr (open->share);
+  gfc_free_expr (open->cc);
   free (open);
 }
 
 
-/* Resolve everything in a gfc_open structure.  */
-
+static int
+compare_to_allowed_values (const char *specifier, const char *allowed[],
+                          const char *allowed_f2003[],
+                          const char *allowed_gnu[], gfc_char_t *value,
+                          const char *statement, bool warn, locus *where,
+                          int *num = NULL);
+
+
+static bool
+check_open_constraints (gfc_open *open, locus *where);
+
+/* Resolve everything in a gfc_open structure.  */
+
 bool
-gfc_resolve_open (gfc_open *open)
+gfc_resolve_open (gfc_open *open, locus *where)
 {
-
   RESOLVE_TAG (&tag_unit, open->unit);
   RESOLVE_TAG (&tag_iomsg, open->iomsg);
   RESOLVE_TAG (&tag_iostat, open->iostat);
@@ -1710,11 +2040,13 @@ gfc_resolve_open (gfc_open *open)
   RESOLVE_TAG (&tag_e_sign, open->sign);
   RESOLVE_TAG (&tag_convert, open->convert);
   RESOLVE_TAG (&tag_newunit, open->newunit);
+  RESOLVE_TAG (&tag_e_share, open->share);
+  RESOLVE_TAG (&tag_cc, open->cc);
 
   if (!gfc_reference_st_label (open->err, ST_LABEL_TARGET))
     return false;
 
-  return true;
+  return check_open_constraints (open, where);
 }
 
 
@@ -1722,11 +2054,13 @@ gfc_resolve_open (gfc_open *open)
    allowed in F95 or F2003, issuing an error message and returning a zero
    value if it is not allowed.  */
 
+
 static int
 compare_to_allowed_values (const char *specifier, const char *allowed[],
-                          const char *allowed_f2003[], 
+                          const char *allowed_f2003[],
                           const char *allowed_gnu[], gfc_char_t *value,
-                          const char *statement, bool warn)
+                          const char *statement, bool warn, locus *where,
+                          int *num)
 {
   int i;
   unsigned int len;
@@ -1743,7 +2077,14 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
   for (i = 0; allowed[i]; i++)
     if (len == strlen (allowed[i])
        && gfc_wide_strncasecmp (value, allowed[i], strlen (allowed[i])) == 0)
+      {
+       if (num)
+         *num = i;
       return 1;
+      }
+
+  if (!where)
+    where = &gfc_current_locus;
 
   for (i = 0; allowed_f2003 && allowed_f2003[i]; i++)
     if (len == strlen (allowed_f2003[i])
@@ -1754,8 +2095,8 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
 
        if (n == WARNING || (warn && n == ERROR))
          {
-           gfc_warning (0, "Fortran 2003: %s specifier in %s statement at %C "
-                        "has value %qs", specifier, statement,
+           gfc_warning (0, "Fortran 2003: %s specifier in %s statement at %L "
+                        "has value %qs", specifier, statement, where,
                         allowed_f2003[i]);
            return 1;
          }
@@ -1763,8 +2104,8 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
          if (n == ERROR)
            {
              gfc_notify_std (GFC_STD_F2003, "%s specifier in "
-                             "%s statement at %C has value %qs", specifier,
-                             statement, allowed_f2003[i]);
+                             "%s statement at %L has value %qs", specifier,
+                             statement, where, allowed_f2003[i]);
              return 0;
            }
 
@@ -1781,8 +2122,8 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
 
        if (n == WARNING || (warn && n == ERROR))
          {
-           gfc_warning (0, "Extension: %s specifier in %s statement at %C "
-                        "has value %qs", specifier, statement,
+           gfc_warning (0, "Extension: %s specifier in %s statement at %L "
+                        "has value %qs", specifier, statement, where,
                         allowed_gnu[i]);
            return 1;
          }
@@ -1790,8 +2131,8 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
          if (n == ERROR)
            {
              gfc_notify_std (GFC_STD_GNU, "%s specifier in "
-                             "%s statement at %C has value %qs", specifier,
-                             statement, allowed_gnu[i]);
+                             "%s statement at %L has value %qs", specifier,
+                             statement, where, allowed_gnu[i]);
              return 0;
            }
 
@@ -1803,98 +2144,42 @@ compare_to_allowed_values (const char *specifier, const char *allowed[],
     {
       char *s = gfc_widechar_to_char (value, -1);
       gfc_warning (0,
-                  "%s specifier in %s statement at %C has invalid value %qs",
-                  specifier, statement, s);
+                  "%s specifier in %s statement at %L has invalid value %qs",
+                  specifier, statement, where, s);
       free (s);
       return 1;
     }
   else
     {
       char *s = gfc_widechar_to_char (value, -1);
-      gfc_error ("%s specifier in %s statement at %C has invalid value %qs",
-                specifier, statement, s);
+      gfc_error ("%s specifier in %s statement at %L has invalid value %qs",
+                specifier, statement, where, s);
       free (s);
       return 0;
     }
 }
 
 
-/* Match an OPEN statement.  */
+/* Check constraints on the OPEN statement.
+   Similar to check_io_constraints for data transfer statements.
+   At this point all tags have already been resolved via resolve_tag, which,
+   among other things, verifies that BT_CHARACTER tags are of default kind.  */
 
-match
-gfc_match_open (void)
+static bool
+check_open_constraints (gfc_open *open, locus *where)
 {
-  gfc_open *open;
-  match m;
-  bool warn;
-
-  m = gfc_match_char ('(');
-  if (m == MATCH_NO)
-    return m;
-
-  open = XCNEW (gfc_open);
-
-  m = match_open_element (open);
-
-  if (m == MATCH_ERROR)
-    goto cleanup;
-  if (m == MATCH_NO)
-    {
-      m = gfc_match_expr (&open->unit);
-      if (m == MATCH_ERROR)
-       goto cleanup;
-    }
-
-  for (;;)
-    {
-      if (gfc_match_char (')') == MATCH_YES)
-       break;
-      if (gfc_match_char (',') != MATCH_YES)
-       goto syntax;
-
-      m = match_open_element (open);
-      if (m == MATCH_ERROR)
-       goto cleanup;
-      if (m == MATCH_NO)
-       goto syntax;
-    }
-
-  if (gfc_match_eos () == MATCH_NO)
-    goto syntax;
-
-  if (gfc_pure (NULL))
-    {
-      gfc_error ("OPEN statement not allowed in PURE procedure at %C");
-      goto cleanup;
-    }
-
-  gfc_unset_implicit_pure (NULL);
-
-  warn = (open->err || open->iostat) ? true : false;
-
-  /* Checks on NEWUNIT specifier.  */
-  if (open->newunit)
-    {
-      if (open->unit)
-       {
-         gfc_error ("UNIT specifier not allowed with NEWUNIT at %C");
-         goto cleanup;
-       }
+#define warn_or_error(...) \
+{ \
+  if (warn) \
+    gfc_warning (0, __VA_ARGS__); \
+  else \
+    { \
+      gfc_error (__VA_ARGS__); \
+      return false; \
+    } \
+}
 
-      if (!(open->file || (open->status
-          && gfc_wide_strncasecmp (open->status->value.character.string,
-                                  "scratch", 7) == 0)))
-       {
-         gfc_error ("NEWUNIT specifier must have FILE= "
-                    "or STATUS='scratch' at %C");
-         goto cleanup;
-       }
-    }
-  else if (!open->unit)
-    {
-      gfc_error ("OPEN statement at %C must have UNIT or NEWUNIT specified");
-      goto cleanup;
-    }
+  bool warn = (open->err || open->iostat) ? true : false;
 
   /* Checks on the ACCESS specifier.  */
   if (open->access && open->access->expr_type == EXPR_CONSTANT)
@@ -1903,39 +2188,47 @@ gfc_match_open (void)
       static const char *access_f2003[] = { "STREAM", NULL };
       static const char *access_gnu[] = { "APPEND", NULL };
 
-      if (!is_char_type ("ACCESS", open->access))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("ACCESS", access_f95, access_f2003,
                                      access_gnu,
                                      open->access->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     "OPEN", warn, &open->access->where))
+       return false;
     }
 
   /* Checks on the ACTION specifier.  */
   if (open->action && open->action->expr_type == EXPR_CONSTANT)
     {
+      gfc_char_t *str = open->action->value.character.string;
       static const char *action[] = { "READ", "WRITE", "READWRITE", NULL };
 
-      if (!is_char_type ("ACTION", open->action))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("ACTION", action, NULL, NULL,
-                                     open->action->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     str, "OPEN", warn, &open->action->where))
+       return false;
+
+      /* With READONLY, only allow ACTION='READ'.  */
+      if (open->readonly && (gfc_wide_strlen (str) != 4
+                            || gfc_wide_strncasecmp (str, "READ", 4) != 0))
+       {
+         gfc_error ("ACTION type conflicts with READONLY specifier at %L",
+                    &open->action->where);
+         return false;
+       }
+    }
+
+  /* If we see READONLY and no ACTION, set ACTION='READ'.  */
+  else if (open->readonly && open->action == NULL)
+    {
+      open->action = gfc_get_character_expr (gfc_default_character_kind,
+                                            &gfc_current_locus, "read", 4);
     }
 
   /* Checks on the ASYNCHRONOUS specifier.  */
   if (open->asynchronous)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "ASYNCHRONOUS= at %C "
-                          "not allowed in Fortran 95"))
-       goto cleanup;
-
-      if (!is_char_type ("ASYNCHRONOUS", open->asynchronous))
-       goto cleanup;
+      if (!gfc_notify_std (GFC_STD_F2003, "ASYNCHRONOUS= at %L "
+                          "not allowed in Fortran 95",
+                          &open->asynchronous->where))
+       return false;
 
       if (open->asynchronous->expr_type == EXPR_CONSTANT)
        {
@@ -1943,20 +2236,17 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("ASYNCHRONOUS", asynchronous,
                        NULL, NULL, open->asynchronous->value.character.string,
-                       "OPEN", warn))
-           goto cleanup;
+                       "OPEN", warn, &open->asynchronous->where))
+           return false;
        }
     }
 
   /* Checks on the BLANK specifier.  */
   if (open->blank)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %C "
-                          "not allowed in Fortran 95"))
-       goto cleanup;
-
-      if (!is_char_type ("BLANK", open->blank))
-       goto cleanup;
+      if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %L "
+                          "not allowed in Fortran 95", &open->blank->where))
+       return false;
 
       if (open->blank->expr_type == EXPR_CONSTANT)
        {
@@ -1964,20 +2254,27 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("BLANK", blank, NULL, NULL,
                                          open->blank->value.character.string,
-                                         "OPEN", warn))
-           goto cleanup;
+                                         "OPEN", warn, &open->blank->where))
+           return false;
        }
     }
 
+  /* Checks on the CARRIAGECONTROL specifier.  */
+  if (open->cc && open->cc->expr_type == EXPR_CONSTANT)
+    {
+      static const char *cc[] = { "LIST", "FORTRAN", "NONE", NULL };
+      if (!compare_to_allowed_values ("CARRIAGECONTROL", cc, NULL, NULL,
+                                     open->cc->value.character.string,
+                                     "OPEN", warn, &open->cc->where))
+       return false;
+    }
+
   /* Checks on the DECIMAL specifier.  */
   if (open->decimal)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %C "
-                          "not allowed in Fortran 95"))
-       goto cleanup;
-
-      if (!is_char_type ("DECIMAL", open->decimal))
-       goto cleanup;
+      if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %L "
+                          "not allowed in Fortran 95", &open->decimal->where))
+       return false;
 
       if (open->decimal->expr_type == EXPR_CONSTANT)
        {
@@ -1985,8 +2282,8 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("DECIMAL", decimal, NULL, NULL,
                                          open->decimal->value.character.string,
-                                         "OPEN", warn))
-           goto cleanup;
+                                         "OPEN", warn, &open->decimal->where))
+           return false;
        }
     }
 
@@ -1997,25 +2294,19 @@ gfc_match_open (void)
        {
          static const char *delim[] = { "APOSTROPHE", "QUOTE", "NONE", NULL };
 
-       if (!is_char_type ("DELIM", open->delim))
-         goto cleanup;
-
          if (!compare_to_allowed_values ("DELIM", delim, NULL, NULL,
                                          open->delim->value.character.string,
-                                         "OPEN", warn))
-         goto cleanup;
+                                         "OPEN", warn, &open->delim->where))
+           return false;
        }
     }
 
   /* Checks on the ENCODING specifier.  */
   if (open->encoding)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "ENCODING= at %C "
-                          "not allowed in Fortran 95"))
-       goto cleanup;
-
-      if (!is_char_type ("ENCODING", open->encoding))
-       goto cleanup;
+      if (!gfc_notify_std (GFC_STD_F2003, "ENCODING= at %L "
+                          "not allowed in Fortran 95", &open->encoding->where))
+       return false;
 
       if (open->encoding->expr_type == EXPR_CONSTANT)
        {
@@ -2023,8 +2314,8 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("ENCODING", encoding, NULL, NULL,
                                          open->encoding->value.character.string,
-                                         "OPEN", warn))
-         goto cleanup;
+                                         "OPEN", warn, &open->encoding->where))
+           return false;
        }
     }
 
@@ -2033,13 +2324,10 @@ gfc_match_open (void)
     {
       static const char *form[] = { "FORMATTED", "UNFORMATTED", NULL };
 
-      if (!is_char_type ("FORM", open->form))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("FORM", form, NULL, NULL,
                                      open->form->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     "OPEN", warn, &open->form->where))
+       return false;
     }
 
   /* Checks on the PAD specifier.  */
@@ -2047,13 +2335,10 @@ gfc_match_open (void)
     {
       static const char *pad[] = { "YES", "NO", NULL };
 
-      if (!is_char_type ("PAD", open->pad))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("PAD", pad, NULL, NULL,
                                      open->pad->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     "OPEN", warn, &open->pad->where))
+       return false;
     }
 
   /* Checks on the POSITION specifier.  */
@@ -2061,24 +2346,18 @@ gfc_match_open (void)
     {
       static const char *position[] = { "ASIS", "REWIND", "APPEND", NULL };
 
-      if (!is_char_type ("POSITION", open->position))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("POSITION", position, NULL, NULL,
                                      open->position->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     "OPEN", warn, &open->position->where))
+       return false;
     }
 
   /* Checks on the ROUND specifier.  */
   if (open->round)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %C "
-                          "not allowed in Fortran 95"))
-      goto cleanup;
-
-      if (!is_char_type ("ROUND", open->round))
-       goto cleanup;
+      if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %L "
+                          "not allowed in Fortran 95", &open->round->where))
+       return false;
 
       if (open->round->expr_type == EXPR_CONSTANT)
        {
@@ -2088,20 +2367,27 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("ROUND", round, NULL, NULL,
                                          open->round->value.character.string,
-                                         "OPEN", warn))
-         goto cleanup;
+                                         "OPEN", warn, &open->round->where))
+           return false;
        }
     }
 
-  /* Checks on the SIGN specifier.  */
-  if (open->sign) 
+  /* Checks on the SHARE specifier.  */
+  if (open->share && open->share->expr_type == EXPR_CONSTANT)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "SIGN= at %C "
-                          "not allowed in Fortran 95"))
-       goto cleanup;
+      static const char *share[] = { "DENYNONE", "DENYRW", NULL };
+      if (!compare_to_allowed_values ("SHARE", share, NULL, NULL,
+                                     open->share->value.character.string,
+                                     "OPEN", warn, &open->share->where))
+       return false;
+    }
 
-      if (!is_char_type ("SIGN", open->sign))
-       goto cleanup;
+  /* Checks on the SIGN specifier.  */
+  if (open->sign)
+    {
+      if (!gfc_notify_std (GFC_STD_F2003, "SIGN= at %L "
+                          "not allowed in Fortran 95", &open->sign->where))
+       return false;
 
       if (open->sign->expr_type == EXPR_CONSTANT)
        {
@@ -2110,28 +2396,18 @@ gfc_match_open (void)
 
          if (!compare_to_allowed_values ("SIGN", sign, NULL, NULL,
                                          open->sign->value.character.string,
-                                         "OPEN", warn))
-         goto cleanup;
+                                         "OPEN", warn, &open->sign->where))
+           return false;
        }
     }
 
-#define warn_or_error(...) \
-{ \
-  if (warn) \
-    gfc_warning (0, __VA_ARGS__); \
-  else \
-    { \
-      gfc_error (__VA_ARGS__); \
-      goto cleanup; \
-    } \
-}
-
   /* Checks on the RECL specifier.  */
   if (open->recl && open->recl->expr_type == EXPR_CONSTANT
       && open->recl->ts.type == BT_INTEGER
       && mpz_sgn (open->recl->value.integer) != 1)
     {
-      warn_or_error ("RECL in OPEN statement at %C must be positive");
+      warn_or_error (G_("RECL in OPEN statement at %L must be positive"),
+                    &open->recl->where);
     }
 
   /* Checks on the STATUS specifier.  */
@@ -2140,13 +2416,10 @@ gfc_match_open (void)
       static const char *status[] = { "OLD", "NEW", "SCRATCH",
        "REPLACE", "UNKNOWN", NULL };
 
-      if (!is_char_type ("STATUS", open->status))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("STATUS", status, NULL, NULL,
                                      open->status->value.character.string,
-                                     "OPEN", warn))
-       goto cleanup;
+                                     "OPEN", warn, &open->status->where))
+       return false;
 
       /* F2003, 9.4.5: If the STATUS= specifier has the value NEW or REPLACE,
         the FILE= specifier shall appear.  */
@@ -2158,8 +2431,9 @@ gfc_match_open (void)
        {
          char *s = gfc_widechar_to_char (open->status->value.character.string,
                                          -1);
-         warn_or_error ("The STATUS specified in OPEN statement at %C is "
-                        "%qs and no FILE specifier is present", s);
+         warn_or_error (G_("The STATUS specified in OPEN statement at %L is "
+                        "%qs and no FILE specifier is present"),
+                        &open->status->where, s);
          free (s);
        }
 
@@ -2168,12 +2442,40 @@ gfc_match_open (void)
       if (gfc_wide_strncasecmp (open->status->value.character.string,
                                "scratch", 7) == 0 && open->file)
        {
-         warn_or_error ("The STATUS specified in OPEN statement at %C "
+         warn_or_error (G_("The STATUS specified in OPEN statement at %L "
                         "cannot have the value SCRATCH if a FILE specifier "
-                        "is present");
+                        "is present"), &open->status->where);
        }
     }
 
+  /* Checks on NEWUNIT specifier.  */
+  if (open->newunit)
+    {
+      if (open->unit)
+       {
+         gfc_error ("UNIT specifier not allowed with NEWUNIT at %L",
+                    &open->newunit->where);
+         return false;
+       }
+
+      if (!open->file &&
+         (!open->status ||
+          (open->status->expr_type == EXPR_CONSTANT
+            && gfc_wide_strncasecmp (open->status->value.character.string,
+                                     "scratch", 7) != 0)))
+       {
+            gfc_error ("NEWUNIT specifier must have FILE= "
+                       "or STATUS='scratch' at %L", &open->newunit->where);
+            return false;
+       }
+    }
+  else if (!open->unit)
+    {
+      gfc_error ("OPEN statement at %L must have UNIT or NEWUNIT specified",
+                where);
+      return false;
+    }
+
   /* Things that are not allowed for unformatted I/O.  */
   if (open->form && open->form->expr_type == EXPR_CONSTANT
       && (open->delim || open->decimal || open->encoding || open->round
@@ -2181,20 +2483,39 @@ gfc_match_open (void)
       && gfc_wide_strncasecmp (open->form->value.character.string,
                               "unformatted", 11) == 0)
     {
-      const char *spec = (open->delim ? "DELIM "
-                                     : (open->pad ? "PAD " : open->blank
-                                                           ? "BLANK " : ""));
+      locus *loc;
+      const char *spec;
+      if (open->delim)
+       {
+         loc = &open->delim->where;
+         spec = "DELIM ";
+       }
+      else if (open->pad)
+       {
+         loc = &open->pad->where;
+         spec = "PAD ";
+       }
+      else if (open->blank)
+       {
+         loc = &open->blank->where;
+         spec = "BLANK ";
+       }
+      else
+       {
+         loc = where;
+         spec = "";
+       }
 
-      warn_or_error ("%s specifier at %C not allowed in OPEN statement for "
-                    "unformatted I/O", spec);
+      warn_or_error (G_("%s specifier at %L not allowed in OPEN statement for "
+                    "unformatted I/O"), spec, loc);
     }
 
   if (open->recl && open->access && open->access->expr_type == EXPR_CONSTANT
       && gfc_wide_strncasecmp (open->access->value.character.string,
                               "stream", 6) == 0)
     {
-      warn_or_error ("RECL specifier not allowed in OPEN statement at %C for "
-                    "stream I/O");
+      warn_or_error (G_("RECL specifier not allowed in OPEN statement at %L for "
+                    "stream I/O"), &open->recl->where);
     }
 
   if (open->position
@@ -2206,11 +2527,64 @@ gfc_match_open (void)
           || gfc_wide_strncasecmp (open->access->value.character.string,
                                    "append", 6) == 0))
     {
-      warn_or_error ("POSITION specifier in OPEN statement at %C only allowed "
-                    "for stream or sequential ACCESS");
+      warn_or_error (G_("POSITION specifier in OPEN statement at %L only allowed "
+                    "for stream or sequential ACCESS"), &open->position->where);
     }
 
+  return true;
 #undef warn_or_error
+}
+
+
+/* Match an OPEN statement.  */
+
+match
+gfc_match_open (void)
+{
+  gfc_open *open;
+  match m;
+
+  m = gfc_match_char ('(');
+  if (m == MATCH_NO)
+    return m;
+
+  open = XCNEW (gfc_open);
+
+  m = match_open_element (open);
+
+  if (m == MATCH_ERROR)
+    goto cleanup;
+  if (m == MATCH_NO)
+    {
+      m = gfc_match_expr (&open->unit);
+      if (m == MATCH_ERROR)
+       goto cleanup;
+    }
+
+  for (;;)
+    {
+      if (gfc_match_char (')') == MATCH_YES)
+       break;
+      if (gfc_match_char (',') != MATCH_YES)
+       goto syntax;
+
+      m = match_open_element (open);
+      if (m == MATCH_ERROR)
+       goto cleanup;
+      if (m == MATCH_NO)
+       goto syntax;
+    }
+
+  if (gfc_match_eos () == MATCH_NO)
+    goto syntax;
+
+  if (gfc_pure (NULL))
+    {
+      gfc_error ("OPEN statement not allowed in PURE procedure at %C");
+      goto cleanup;
+    }
+
+  gfc_unset_implicit_pure (NULL);
 
   new_st.op = EXEC_OPEN;
   new_st.ext.open = open;
@@ -2255,8 +2629,6 @@ match_close_element (gfc_close *close)
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_iomsg, &close->iomsg);
-  if (m == MATCH_YES && !check_char_variable (close->iomsg))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
   m = match_out_tag (&tag_iostat, &close->iostat);
@@ -2277,7 +2649,6 @@ gfc_match_close (void)
 {
   gfc_close *close;
   match m;
-  bool warn;
 
   m = gfc_match_char ('(');
   if (m == MATCH_NO)
@@ -2323,39 +2694,56 @@ gfc_match_close (void)
 
   gfc_unset_implicit_pure (NULL);
 
-  warn = (close->iostat || close->err) ? true : false;
+  new_st.op = EXEC_CLOSE;
+  new_st.ext.close = close;
+  return MATCH_YES;
+
+syntax:
+  gfc_syntax_error (ST_CLOSE);
+
+cleanup:
+  gfc_free_close (close);
+  return MATCH_ERROR;
+}
+
+
+static bool
+check_close_constraints (gfc_close *close, locus *where)
+{
+  bool warn = (close->iostat || close->err) ? true : false;
+
+  if (close->unit == NULL)
+    {
+      gfc_error ("CLOSE statement at %L requires a UNIT number", where);
+      return false;
+    }
+
+  if (close->unit->expr_type == EXPR_CONSTANT
+      && close->unit->ts.type == BT_INTEGER
+      && mpz_sgn (close->unit->value.integer) < 0)
+    {
+      gfc_error ("UNIT number in CLOSE statement at %L must be non-negative",
+                &close->unit->where);
+    }
 
   /* Checks on the STATUS specifier.  */
   if (close->status && close->status->expr_type == EXPR_CONSTANT)
     {
       static const char *status[] = { "KEEP", "DELETE", NULL };
 
-      if (!is_char_type ("STATUS", close->status))
-       goto cleanup;
-
       if (!compare_to_allowed_values ("STATUS", status, NULL, NULL,
                                      close->status->value.character.string,
-                                     "CLOSE", warn))
-       goto cleanup;
+                                     "CLOSE", warn, &close->status->where))
+       return false;
     }
 
-  new_st.op = EXEC_CLOSE;
-  new_st.ext.close = close;
-  return MATCH_YES;
-
-syntax:
-  gfc_syntax_error (ST_CLOSE);
-
-cleanup:
-  gfc_free_close (close);
-  return MATCH_ERROR;
+  return true;
 }
 
-
 /* Resolve everything in a gfc_close structure.  */
 
 bool
-gfc_resolve_close (gfc_close *close)
+gfc_resolve_close (gfc_close *close, locus *where)
 {
   RESOLVE_TAG (&tag_unit, close->unit);
   RESOLVE_TAG (&tag_iomsg, close->iomsg);
@@ -2365,33 +2753,7 @@ gfc_resolve_close (gfc_close *close)
   if (!gfc_reference_st_label (close->err, ST_LABEL_TARGET))
     return false;
 
-  if (close->unit == NULL)
-    {
-      /* Find a locus from one of the arguments to close, when UNIT is
-        not specified.  */
-      locus loc = gfc_current_locus;
-      if (close->status)
-       loc = close->status->where;
-      else if (close->iostat)
-       loc = close->iostat->where;
-      else if (close->iomsg)
-       loc = close->iomsg->where;
-      else if (close->err)
-       loc = close->err->where;
-
-      gfc_error ("CLOSE statement at %L requires a UNIT number", &loc);
-      return false;
-    }
-
-  if (close->unit->expr_type == EXPR_CONSTANT
-      && close->unit->ts.type == BT_INTEGER
-      && mpz_sgn (close->unit->value.integer) < 0)
-    {
-      gfc_error ("UNIT number in CLOSE statement at %L must be non-negative",
-                &close->unit->where);
-    }
-
-  return true;
+  return check_close_constraints (close, where);
 }
 
 
@@ -2418,8 +2780,6 @@ match_file_element (gfc_filepos *fp)
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_iomsg, &fp->iomsg);
-  if (m == MATCH_YES && !check_char_variable (fp->iomsg))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
   m = match_out_tag (&tag_iostat, &fp->iostat);
@@ -2457,7 +2817,7 @@ match_filepos (gfc_statement st, gfc_exec_op op)
 
   m = match_file_element (fp);
   if (m == MATCH_ERROR)
-    goto done;
+    goto cleanup;
   if (m == MATCH_NO)
     {
       m = gfc_match_expr (&fp->unit);
@@ -2507,11 +2867,18 @@ cleanup:
 
 
 bool
-gfc_resolve_filepos (gfc_filepos *fp)
+gfc_resolve_filepos (gfc_filepos *fp, locus *where)
 {
   RESOLVE_TAG (&tag_unit, fp->unit);
   RESOLVE_TAG (&tag_iostat, fp->iostat);
   RESOLVE_TAG (&tag_iomsg, fp->iomsg);
+
+  if (!fp->unit && (fp->iostat || fp->iomsg || fp->err))
+    {
+      gfc_error ("UNIT number missing in statement at %L", where);
+      return false;
+    }
+
   if (!gfc_reference_st_label (fp->err, ST_LABEL_TARGET))
     return false;
 
@@ -2521,6 +2888,7 @@ gfc_resolve_filepos (gfc_filepos *fp)
     {
       gfc_error ("UNIT number in statement at %L must be non-negative",
                 &fp->unit->where);
+      return false;
     }
 
   return true;
@@ -2581,6 +2949,7 @@ static match
 match_dt_unit (io_kind k, gfc_dt *dt)
 {
   gfc_expr *e;
+  char c;
 
   if (gfc_match_char ('*') == MATCH_YES)
     {
@@ -2588,6 +2957,11 @@ match_dt_unit (io_kind k, gfc_dt *dt)
        goto conflict;
 
       dt->io_unit = default_unit (k);
+
+      c = gfc_peek_ascii_char ();
+      if (c == ')')
+       gfc_error_now ("Missing format with default unit at %C");
+
       return MATCH_YES;
     }
 
@@ -2684,6 +3058,30 @@ conflict:
   return MATCH_ERROR;
 }
 
+/* Check for formatted read and write DTIO procedures.  */
+
+static bool
+dtio_procs_present (gfc_symbol *sym, io_kind k)
+{
+  gfc_symbol *derived;
+
+  if (sym && sym->ts.u.derived)
+    {
+      if (sym->ts.type == BT_CLASS && CLASS_DATA (sym))
+       derived = CLASS_DATA (sym)->ts.u.derived;
+      else if (sym->ts.type == BT_DERIVED)
+       derived = sym->ts.u.derived;
+      else
+       return false;
+      if ((k == M_WRITE || k == M_PRINT) && 
+         (gfc_find_specific_dtio_proc (derived, true, true) != NULL))
+       return true;
+      if ((k == M_READ) &&
+         (gfc_find_specific_dtio_proc (derived, false, true) != NULL))
+       return true;
+    }
+  return false;
+}
 
 /* Traverse a namelist that is part of a READ statement to make sure
    that none of the variables in the namelist are INTENT(IN).  Returns
@@ -2755,8 +3153,6 @@ match_dt_element (io_kind k, gfc_dt *dt)
     }
 
   m = match_etag (&tag_e_async, &dt->asynchronous);
-  if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", dt->asynchronous))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_e_blank, &dt->blank);
@@ -2787,8 +3183,6 @@ match_dt_element (io_kind k, gfc_dt *dt)
   if (m != MATCH_NO)
     return m;
   m = match_etag (&tag_iomsg, &dt->iomsg);
-  if (m == MATCH_YES && !check_char_variable (dt->iomsg))
-    return MATCH_ERROR;
   if (m != MATCH_NO)
     return m;
 
@@ -2858,10 +3252,17 @@ gfc_free_dt (gfc_dt *dt)
 }
 
 
+static const char *
+io_kind_name (io_kind k);
+
+static bool
+check_io_constraints (io_kind k, gfc_dt *dt, gfc_code *io_code,
+                     locus *spec_end);
+
 /* Resolve everything in a gfc_dt structure.  */
 
 bool
-gfc_resolve_dt (gfc_dt *dt, locus *loc)
+gfc_resolve_dt (gfc_code *dt_code, gfc_dt *dt, locus *loc)
 {
   gfc_expr *e;
   io_kind k;
@@ -2886,6 +3287,18 @@ gfc_resolve_dt (gfc_dt *dt, locus *loc)
   RESOLVE_TAG (&tag_e_decimal, dt->decimal);
   RESOLVE_TAG (&tag_e_async, dt->asynchronous);
 
+  /* Check I/O constraints.
+     To validate NAMELIST we need to check if we were also given an I/O list,
+     which is stored in code->block->next with op EXEC_TRANSFER.
+     Note that the I/O list was already resolved from resolve_transfer.  */
+  gfc_code *io_code = NULL;
+  if (dt_code && dt_code->block && dt_code->block->next
+      && dt_code->block->next->op == EXEC_TRANSFER)
+    io_code = dt_code->block->next;
+
+  if (!check_io_constraints (k, dt, io_code, loc))
+    return false;
+
   e = dt->io_unit;
   if (e == NULL)
     {
@@ -2893,6 +3306,14 @@ gfc_resolve_dt (gfc_dt *dt, locus *loc)
       return false;
     }
 
+  if (e->symtree && e->symtree->n.sym->attr.flavor == FL_PARAMETER
+      && e->ts.type == BT_CHARACTER)
+    {
+      gfc_error ("UNIT specification at %L must "
+      "not be a character PARAMETER", &e->where);
+      return false;
+    }
+
   if (gfc_resolve_expr (e)
       && (e->ts.type != BT_INTEGER
          && (e->ts.type != BT_CHARACTER || e->expr_type != EXPR_VARIABLE)))
@@ -2962,7 +3383,7 @@ gfc_resolve_dt (gfc_dt *dt, locus *loc)
 
   /* If we are reading and have a namelist, check that all namelist symbols
      can appear in a variable definition context.  */
-  if (k == M_READ && dt->namelist)
+  if (dt->namelist)
     {
       gfc_namelist* n;
       for (n = dt->namelist->namelist; n; n = n->next)
@@ -2970,23 +3391,56 @@ gfc_resolve_dt (gfc_dt *dt, locus *loc)
          gfc_expr* e;
          bool t;
 
-         e = gfc_get_variable_expr (gfc_find_sym_in_symtree (n->sym));
-         t = gfc_check_vardef_context (e, false, false, false, NULL);
-         gfc_free_expr (e);
+         if (k == M_READ)
+           {
+             e = gfc_get_variable_expr (gfc_find_sym_in_symtree (n->sym));
+             t = gfc_check_vardef_context (e, false, false, false, NULL);
+             gfc_free_expr (e);
+    
+             if (!t)
+               {
+                 gfc_error ("NAMELIST %qs in READ statement at %L contains"
+                            " the symbol %qs which may not appear in a"
+                            " variable definition context",
+                            dt->namelist->name, loc, n->sym->name);
+                 return false;
+               }
+           }
+
+         t = dtio_procs_present (n->sym, k);
 
-         if (!t)
+         if (n->sym->ts.type == BT_CLASS && !t)
            {
-             gfc_error ("NAMELIST %qs in READ statement at %L contains"
-                        " the symbol %qs which may not appear in a"
-                        " variable definition context",
-                        dt->namelist->name, loc, n->sym->name);
+             gfc_error ("NAMELIST object %qs in namelist %qs at %L is "
+                        "polymorphic and requires a defined input/output "
+                        "procedure", n->sym->name, dt->namelist->name, loc);
              return false;
            }
+    
+         if ((n->sym->ts.type == BT_DERIVED)
+             && (n->sym->ts.u.derived->attr.alloc_comp
+                 || n->sym->ts.u.derived->attr.pointer_comp))
+           {
+             if (!gfc_notify_std (GFC_STD_F2003, "NAMELIST object %qs in "
+                                  "namelist %qs at %L with ALLOCATABLE "
+                                  "or POINTER components", n->sym->name,
+                                  dt->namelist->name, loc))
+               return false;
+    
+             if (!t)
+               {
+                 gfc_error ("NAMELIST object %qs in namelist %qs at %L has "
+                            "ALLOCATABLE or POINTER components and thus requires "
+                            "a defined input/output procedure", n->sym->name,
+                            dt->namelist->name, loc);
+                 return false;
+               }
+           }
        }
     }
 
   if (dt->extra_comma
-      && !gfc_notify_std (GFC_STD_GNU, "Comma before i/o item list at %L", 
+      && !gfc_notify_std (GFC_STD_LEGACY, "Comma before i/o item list at %L", 
                          &dt->extra_comma->where))
     return false;
 
@@ -3031,7 +3485,7 @@ gfc_resolve_dt (gfc_dt *dt, locus *loc)
       && dt->format_label->defined == ST_LABEL_UNKNOWN)
     {
       gfc_error ("FORMAT label %d at %L not defined", dt->format_label->value,
-                &dt->format_label->where);
+                loc);
       return false;
     }
 
@@ -3181,7 +3635,26 @@ match_io_element (io_kind k, gfc_code **cpp)
     {
       m = gfc_match_variable (&expr, 0);
       if (m == MATCH_NO)
-       gfc_error ("Expected variable in READ statement at %C");
+       {
+         gfc_error ("Expecting variable in READ statement at %C");
+         m = MATCH_ERROR;
+       }
+
+      if (m == MATCH_YES && expr->expr_type == EXPR_CONSTANT)
+       {
+         gfc_error ("Expecting variable or io-implied-do in READ statement "
+                  "at %L", &expr->where);
+         m = MATCH_ERROR;
+       }
+
+      if (m == MATCH_YES
+         && expr->expr_type == EXPR_VARIABLE
+         && expr->symtree->n.sym->attr.external)
+       {
+         gfc_error ("Expecting variable or io-implied-do at %L",
+                    &expr->where);
+         m = MATCH_ERROR;
+       }
     }
   else
     {
@@ -3189,6 +3662,15 @@ match_io_element (io_kind k, gfc_code **cpp)
       if (m == MATCH_NO)
        gfc_error ("Expected expression in %s statement at %C",
                   io_kind_name (k));
+
+      if (m == MATCH_YES && expr->ts.type == BT_BOZ)
+       {
+         if (gfc_invalid_boz (G_("BOZ literal constant at %L cannot appear in"
+                               " an output IO list"), &gfc_current_locus))
+           return MATCH_ERROR;
+         if (!gfc_boz2int (expr, gfc_max_integer_kind))
+           return MATCH_ERROR;
+       };
     }
 
   if (m == MATCH_YES && k == M_READ && gfc_check_do_variable (expr->symtree))
@@ -3271,21 +3753,26 @@ terminate_io (gfc_code *io_code)
 
 
 /* Check the constraints for a data transfer statement.  The majority of the
-   constraints appearing in 9.4 of the standard appear here.  Some are handled
-   in resolve_tag and others in gfc_resolve_dt.  */
+   constraints appearing in 9.4 of the standard appear here.
 
-static match
+   Tag expressions are already resolved by resolve_tag, which includes
+   verifying the type, that they are scalar, and verifying that BT_CHARACTER
+   tags are of default kind.  */
+
+static bool
 check_io_constraints (io_kind k, gfc_dt *dt, gfc_code *io_code,
                      locus *spec_end)
 {
-#define io_constraint(condition,msg,arg)\
+#define io_constraint(condition, msg, arg)\
 if (condition) \
   {\
-    gfc_error(msg,arg);\
-    m = MATCH_ERROR;\
+    if ((arg)->lb != NULL)\
+      gfc_error ((msg), (arg));\
+    else\
+      gfc_error ((msg), spec_end);\
+    return false;\
   }
 
-  match m;
   gfc_expr *expr;
   gfc_symbol *sym = NULL;
   bool warn, unformatted;
@@ -3294,8 +3781,6 @@ if (condition) \
   unformatted = dt->format_expr == NULL && dt->format_label == NULL
                && dt->namelist == NULL;
 
-  m = MATCH_YES;
-
   expr = dt->io_unit;
   if (expr && expr->expr_type == EXPR_VARIABLE
       && expr->ts.type == BT_CHARACTER)
@@ -3313,7 +3798,7 @@ if (condition) \
       io_constraint (dt->rec != NULL,
                     "REC tag at %L is incompatible with internal file",
                     &dt->rec->where);
-    
+
       io_constraint (dt->pos != NULL,
                     "POS tag at %L is incompatible with internal file",
                     &dt->pos->where);
@@ -3330,7 +3815,7 @@ if (condition) \
        {
          if (!gfc_notify_std (GFC_STD_F2003, "Internal file at %L with "
                               "namelist", &expr->where))
-           m = MATCH_ERROR;
+           return false;
        }
 
       io_constraint (dt->advance != NULL,
@@ -3341,62 +3826,60 @@ if (condition) \
   if (expr && expr->ts.type != BT_CHARACTER)
     {
 
-      io_constraint (gfc_pure (NULL) && (k == M_READ || k == M_WRITE),
-                    "IO UNIT in %s statement at %C must be "
+      if (gfc_pure (NULL) && (k == M_READ || k == M_WRITE))
+       {
+         gfc_error ("IO UNIT in %s statement at %L must be "
                     "an internal file in a PURE procedure",
-                    io_kind_name (k));
+                    io_kind_name (k), &expr->where);
+         return false;
+       }
 
       if (k == M_READ || k == M_WRITE)
        gfc_unset_implicit_pure (NULL);
     }
 
-  if (k != M_READ)
-    {
-      io_constraint (dt->end, "END tag not allowed with output at %L",
-                    &dt->end_where);
-
-      io_constraint (dt->eor, "EOR tag not allowed with output at %L",
-                    &dt->eor_where);
-
-      io_constraint (dt->blank, "BLANK= specifier not allowed with output at %L",
-                    &dt->blank->where);
-
-      io_constraint (dt->pad, "PAD= specifier not allowed with output at %L",
-                    &dt->pad->where);
-
-      io_constraint (dt->size, "SIZE= specifier not allowed with output at %L",
-                    &dt->size->where);
-    }
-  else
-    {
-      io_constraint (dt->size && dt->advance == NULL,
-                    "SIZE tag at %L requires an ADVANCE tag",
-                    &dt->size->where);
-
-      io_constraint (dt->eor && dt->advance == NULL,
-                    "EOR tag at %L requires an ADVANCE tag",
-                    &dt->eor_where);
-    }
-
-  if (dt->asynchronous) 
+  if (dt->asynchronous)
     {
+      int num = -1;
       static const char * asynchronous[] = { "YES", "NO", NULL };
 
+      /* Note: gfc_reduce_init_expr reports an error if not init-expr.  */
       if (!gfc_reduce_init_expr (dt->asynchronous))
-       {
-         gfc_error ("ASYNCHRONOUS= specifier at %L must be an initialization "
-                    "expression", &dt->asynchronous->where);
-         return MATCH_ERROR;
-       }
-
-      if (!is_char_type ("ASYNCHRONOUS", dt->asynchronous))
-       return MATCH_ERROR;
+       return false;
 
       if (!compare_to_allowed_values
                ("ASYNCHRONOUS", asynchronous, NULL, NULL,
                 dt->asynchronous->value.character.string,
-                io_kind_name (k), warn))
-       return MATCH_ERROR;
+                io_kind_name (k), warn, &dt->asynchronous->where, &num))
+       return false;
+
+      gcc_checking_assert (num != -1);
+
+      /* For "YES", mark related symbols as asynchronous.  */
+      if (num == 0)
+       {
+         /* SIZE variable.  */
+         if (dt->size)
+           dt->size->symtree->n.sym->attr.asynchronous = 1;
+
+         /* Variables in a NAMELIST.  */
+         if (dt->namelist)
+           for (gfc_namelist *nl = dt->namelist->namelist; nl; nl = nl->next)
+             nl->sym->attr.asynchronous = 1;
+
+         /* Variables in an I/O list.  */
+         for (gfc_code *xfer = io_code; xfer && xfer->op == EXEC_TRANSFER;
+              xfer = xfer->next)
+           {
+             gfc_expr *expr = xfer->expr1;
+             while (expr != NULL && expr->expr_type == EXPR_OP
+                    && expr->value.op.op == INTRINSIC_PARENTHESES)
+               expr = expr->value.op.op1;
+
+             if (expr && expr->expr_type == EXPR_VARIABLE)
+               expr->symtree->n.sym->attr.asynchronous = 1;
+           }
+       }
     }
 
   if (dt->id)
@@ -3413,36 +3896,31 @@ if (condition) \
 
   if (dt->decimal)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %C "
-                          "not allowed in Fortran 95"))
-       return MATCH_ERROR;
+      if (!gfc_notify_std (GFC_STD_F2003, "DECIMAL= at %L "
+                          "not allowed in Fortran 95", &dt->decimal->where))
+       return false;
 
       if (dt->decimal->expr_type == EXPR_CONSTANT)
        {
          static const char * decimal[] = { "COMMA", "POINT", NULL };
 
-      if (!is_char_type ("DECIMAL", dt->decimal))
-       return MATCH_ERROR;
-
          if (!compare_to_allowed_values ("DECIMAL", decimal, NULL, NULL,
                                          dt->decimal->value.character.string,
-                                         io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                         io_kind_name (k), warn,
+                                         &dt->decimal->where))
+           return false;
 
          io_constraint (unformatted,
                         "the DECIMAL= specifier at %L must be with an "
                         "explicit format expression", &dt->decimal->where);
        }
     }
-  
+
   if (dt->blank)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %C "
-                          "not allowed in Fortran 95"))
-       return MATCH_ERROR;
-
-      if (!is_char_type ("BLANK", dt->blank))
-       return MATCH_ERROR;
+      if (!gfc_notify_std (GFC_STD_F2003, "BLANK= at %L "
+                          "not allowed in Fortran 95", &dt->blank->where))
+       return false;
 
       if (dt->blank->expr_type == EXPR_CONSTANT)
        {
@@ -3451,8 +3929,9 @@ if (condition) \
 
          if (!compare_to_allowed_values ("BLANK", blank, NULL, NULL,
                                          dt->blank->value.character.string,
-                                         io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                         io_kind_name (k), warn,
+                                         &dt->blank->where))
+           return false;
 
          io_constraint (unformatted,
                         "the BLANK= specifier at %L must be with an "
@@ -3462,12 +3941,9 @@ if (condition) \
 
   if (dt->pad)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "PAD= at %C "
-                          "not allowed in Fortran 95"))
-       return MATCH_ERROR;
-
-      if (!is_char_type ("PAD", dt->pad))
-       return MATCH_ERROR;
+      if (!gfc_notify_std (GFC_STD_F2003, "PAD= at %L "
+                          "not allowed in Fortran 95", &dt->pad->where))
+       return false;
 
       if (dt->pad->expr_type == EXPR_CONSTANT)
        {
@@ -3475,8 +3951,9 @@ if (condition) \
 
          if (!compare_to_allowed_values ("PAD", pad, NULL, NULL,
                                          dt->pad->value.character.string,
-                                         io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                         io_kind_name (k), warn,
+                                         &dt->pad->where))
+           return false;
 
          io_constraint (unformatted,
                         "the PAD= specifier at %L must be with an "
@@ -3486,12 +3963,9 @@ if (condition) \
 
   if (dt->round)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %C "
-                          "not allowed in Fortran 95"))
-       return MATCH_ERROR;
-
-      if (!is_char_type ("ROUND", dt->round))
-       return MATCH_ERROR;
+      if (!gfc_notify_std (GFC_STD_F2003, "ROUND= at %L "
+                          "not allowed in Fortran 95", &dt->round->where))
+       return false;
 
       if (dt->round->expr_type == EXPR_CONSTANT)
        {
@@ -3501,20 +3975,18 @@ if (condition) \
 
          if (!compare_to_allowed_values ("ROUND", round, NULL, NULL,
                                          dt->round->value.character.string,
-                                         io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                         io_kind_name (k), warn,
+                                         &dt->round->where))
+           return false;
        }
     }
-  
+
   if (dt->sign)
     {
       /* When implemented, change the following to use gfc_notify_std F2003.
-      if (gfc_notify_std (GFC_STD_F2003, "SIGN= at %C "
-         "not allowed in Fortran 95") == false)
-       return MATCH_ERROR;  */
-
-      if (!is_char_type ("SIGN", dt->sign))
-       return MATCH_ERROR;
+      if (gfc_notify_std (GFC_STD_F2003, "SIGN= at %L "
+         "not allowed in Fortran 95", &dt->sign->where) == false)
+       return false;  */
 
       if (dt->sign->expr_type == EXPR_CONSTANT)
        {
@@ -3523,8 +3995,8 @@ if (condition) \
 
          if (!compare_to_allowed_values ("SIGN", sign, NULL, NULL,
                                      dt->sign->value.character.string,
-                                     io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                     io_kind_name (k), warn, &dt->sign->where))
+           return false;
 
          io_constraint (unformatted,
                         "SIGN= specifier at %L must be with an "
@@ -3538,12 +4010,9 @@ if (condition) \
 
   if (dt->delim)
     {
-      if (!gfc_notify_std (GFC_STD_F2003, "DELIM= at %C "
-                          "not allowed in Fortran 95"))
-       return MATCH_ERROR;
-
-      if (!is_char_type ("DELIM", dt->delim))
-       return MATCH_ERROR;
+      if (!gfc_notify_std (GFC_STD_F2003, "DELIM= at %L "
+                          "not allowed in Fortran 95", &dt->delim->where))
+       return false;
 
       if (dt->delim->expr_type == EXPR_CONSTANT)
        {
@@ -3551,13 +4020,14 @@ if (condition) \
 
          if (!compare_to_allowed_values ("DELIM", delim, NULL, NULL,
                                          dt->delim->value.character.string,
-                                         io_kind_name (k), warn))
-           return MATCH_ERROR;
+                                         io_kind_name (k), warn,
+                                         &dt->delim->where))
+           return false;
 
          io_constraint (k == M_READ,
                         "DELIM= specifier at %L not allowed in a "
                         "READ statement", &dt->delim->where);
-      
+
          io_constraint (dt->format_label != &format_asterisk
                         && dt->namelist == NULL,
                         "DELIM= specifier at %L must have FMT=*",
@@ -3565,10 +4035,10 @@ if (condition) \
 
          io_constraint (unformatted && dt->namelist == NULL,
                         "DELIM= specifier at %L must be with FMT=* or "
-                        "NML= specifier ", &dt->delim->where);
+                        "NML= specifier", &dt->delim->where);
        }
     }
-  
+
   if (dt->namelist)
     {
       io_constraint (io_code && dt->namelist,
@@ -3645,17 +4115,41 @@ if (condition) \
 
       io_constraint (dt->eor && not_no && k == M_READ,
                     "EOR tag at %L requires an ADVANCE = %<NO%>",
-                    &dt->eor_where);      
+                    &dt->eor_where);
     }
 
-  expr = dt->format_expr;
-  if (!gfc_simplify_expr (expr, 0)
-      || !check_format_string (expr, k == M_READ))
-    return MATCH_ERROR;
+  if (k != M_READ)
+    {
+      io_constraint (dt->end, "END tag not allowed with output at %L",
+                    &dt->end_where);
 
-  return m;
-}
+      io_constraint (dt->eor, "EOR tag not allowed with output at %L",
+                    &dt->eor_where);
+
+      io_constraint (dt->blank,
+                    "BLANK= specifier not allowed with output at %L",
+                    &dt->blank->where);
+
+      io_constraint (dt->pad, "PAD= specifier not allowed with output at %L",
+                    &dt->pad->where);
+
+      io_constraint (dt->size, "SIZE= specifier not allowed with output at %L",
+                    &dt->size->where);
+    }
+  else
+    {
+      io_constraint (dt->size && dt->advance == NULL,
+                    "SIZE tag at %L requires an ADVANCE tag",
+                    &dt->size->where);
+
+      io_constraint (dt->eor && dt->advance == NULL,
+                    "EOR tag at %L requires an ADVANCE tag",
+                    &dt->eor_where);
+    }
+
+  return true;
 #undef io_constraint
+}
 
 
 /* Match a READ, WRITE or PRINT statement.  */
@@ -3668,7 +4162,7 @@ match_io (io_kind k)
   gfc_symbol *sym;
   int comma_flag;
   locus where;
-  locus spec_end;
+  locus control;
   gfc_dt *dt;
   match m;
 
@@ -3704,6 +4198,23 @@ match_io (io_kind k)
              else
                gfc_current_locus = where;
            }
+
+         if (gfc_match_char ('*') == MATCH_YES
+             && gfc_match_char(',') == MATCH_YES)
+           {
+             locus where2 = gfc_current_locus;
+             if (gfc_match_eos () == MATCH_YES)
+               {
+                 gfc_current_locus = where2;
+                 gfc_error ("Comma after * at %C not allowed without I/O list");
+                 m = MATCH_ERROR;
+                 goto cleanup;
+               }
+             else
+               gfc_current_locus = where;
+           }
+         else
+           gfc_current_locus = where;
        }
 
       if (gfc_current_form == FORM_FREE)
@@ -3730,21 +4241,59 @@ match_io (io_kind k)
     {
       /* Before issuing an error for a malformed 'print (1,*)' type of
         error, check for a default-char-expr of the form ('(I0)').  */
-      if (k == M_PRINT && m == MATCH_YES)
-       {
-         /* Reset current locus to get the initial '(' in an expression.  */
-         gfc_current_locus = where;
-         dt->format_expr = NULL;
-         m = match_dt_format (dt);
+      if (m == MATCH_YES)
+        {
+         control = gfc_current_locus;
+         if (k == M_PRINT)
+           {
+             /* Reset current locus to get the initial '(' in an expression.  */
+             gfc_current_locus = where;
+             dt->format_expr = NULL;
+             m = match_dt_format (dt);
 
-         if (m == MATCH_ERROR)
-           goto cleanup;
-         if (m == MATCH_NO || dt->format_expr == NULL)
-           goto syntax;
+             if (m == MATCH_ERROR)
+               goto cleanup;
+             if (m == MATCH_NO || dt->format_expr == NULL)
+               goto syntax;
 
-         comma_flag = 1;
-         dt->io_unit = default_unit (k);
-         goto get_io_list;
+             comma_flag = 1;
+             dt->io_unit = default_unit (k);
+             goto get_io_list;
+           }
+         if (k == M_READ)
+           {
+             /* Commit any pending symbols now so that when we undo
+                symbols later we wont lose them.  */
+             gfc_commit_symbols ();
+             /* Reset current locus to get the initial '(' in an expression.  */
+             gfc_current_locus = where;
+             dt->format_expr = NULL;
+             m = gfc_match_expr (&dt->format_expr);
+             if (m == MATCH_YES)
+               {
+                 if (dt->format_expr
+                     && dt->format_expr->ts.type == BT_CHARACTER)
+                   {
+                     comma_flag = 1;
+                     dt->io_unit = default_unit (k);
+                     goto get_io_list;
+                   }
+                 else
+                   {
+                     gfc_free_expr (dt->format_expr);
+                     dt->format_expr = NULL;
+                     gfc_current_locus = control;
+                   }
+               }
+             else
+               {
+                 gfc_clear_error ();
+                 gfc_undo_symbols ();
+                 gfc_free_expr (dt->format_expr);
+                 dt->format_expr = NULL;
+                 gfc_current_locus = control;
+               }
+           }
        }
     }
 
@@ -3816,9 +4365,6 @@ loop:
 
 get_io_list:
 
-  /* Used in check_io_constraints, where no locus is available.  */
-  spec_end = gfc_current_locus;
-
   /* Save the IO kind for later use.  */
   dt->dt_io_kind = gfc_get_iokind_expr (&gfc_current_locus, k);
 
@@ -3845,12 +4391,16 @@ get_io_list:
        goto syntax;
     }
 
-  /* A full IO statement has been matched.  Check the constraints.  spec_end is
-     supplied for cases where no locus is supplied.  */
-  m = check_io_constraints (k, dt, io_code, &spec_end);
+  /* See if we want to use defaults for missing exponents in real transfers
+     and other DEC runtime extensions. */
+  if (flag_dec_format_defaults)
+    dt->dec_ext = 1;
 
-  if (m == MATCH_ERROR)
-    goto cleanup;
+  /* Check the format string now.  */
+  if (dt->format_expr
+      && (!gfc_simplify_expr (dt->format_expr, 0)
+         || !check_format_string (dt->format_expr, k == M_READ)))
+    return MATCH_ERROR;
 
   new_st.op = (k == M_READ) ? EXEC_READ : EXEC_WRITE;
   new_st.ext.dt = dt;
@@ -3951,6 +4501,8 @@ gfc_free_inquire (gfc_inquire *inquire)
   gfc_free_expr (inquire->sign);
   gfc_free_expr (inquire->size);
   gfc_free_expr (inquire->round);
+  gfc_free_expr (inquire->share);
+  gfc_free_expr (inquire->cc);
   free (inquire);
 }
 
@@ -3968,8 +4520,6 @@ match_inquire_element (gfc_inquire *inquire)
   RETM m = match_etag (&tag_file, &inquire->file);
   RETM m = match_ltag (&tag_err, &inquire->err);
   RETM m = match_etag (&tag_iomsg, &inquire->iomsg);
-  if (m == MATCH_YES && !check_char_variable (inquire->iomsg))
-    return MATCH_ERROR;
   RETM m = match_out_tag (&tag_iostat, &inquire->iostat);
   RETM m = match_vtag (&tag_exist, &inquire->exist);
   RETM m = match_vtag (&tag_opened, &inquire->opened);
@@ -3991,8 +4541,6 @@ match_inquire_element (gfc_inquire *inquire)
   RETM m = match_vtag (&tag_write, &inquire->write);
   RETM m = match_vtag (&tag_readwrite, &inquire->readwrite);
   RETM m = match_vtag (&tag_s_async, &inquire->asynchronous);
-  if (m == MATCH_YES && !is_char_type ("ASYNCHRONOUS", inquire->asynchronous))
-    return MATCH_ERROR;
   RETM m = match_vtag (&tag_s_delim, &inquire->delim);
   RETM m = match_vtag (&tag_s_decimal, &inquire->decimal);
   RETM m = match_out_tag (&tag_size, &inquire->size);
@@ -4006,6 +4554,8 @@ match_inquire_element (gfc_inquire *inquire)
   RETM m = match_vtag (&tag_pending, &inquire->pending);
   RETM m = match_vtag (&tag_id, &inquire->id);
   RETM m = match_vtag (&tag_s_iqstream, &inquire->iqstream);
+  RETM m = match_dec_vtag (&tag_v_share, &inquire->share);
+  RETM m = match_dec_vtag (&tag_v_cc, &inquire->cc);
   RETM return MATCH_NO;
 }
 
@@ -4052,6 +4602,17 @@ gfc_match_inquire (void)
       if (m == MATCH_NO)
        goto syntax;
 
+      for (gfc_code *c = code; c; c = c->next)
+       if (c->expr1 && c->expr1->expr_type == EXPR_FUNCTION
+           && c->expr1->symtree && c->expr1->symtree->n.sym->attr.function
+           && !c->expr1->symtree->n.sym->attr.external
+           && strcmp (c->expr1->symtree->name, "null") == 0)
+         {
+           gfc_error ("NULL() near %L cannot appear in INQUIRE statement",
+                      &c->expr1->where);
+           goto cleanup;
+         }
+
       new_st.op = EXEC_IOLENGTH;
       new_st.expr1 = inquire->iolength;
       new_st.ext.inquire = inquire;
@@ -4111,9 +4672,11 @@ gfc_match_inquire (void)
 
   if (inquire->unit != NULL && inquire->unit->expr_type == EXPR_CONSTANT
       && inquire->unit->ts.type == BT_INTEGER
-      && mpz_get_si (inquire->unit->value.integer) == GFC_INTERNAL_UNIT)
+      && ((mpz_get_si (inquire->unit->value.integer) == GFC_INTERNAL_UNIT4)
+      || (mpz_get_si (inquire->unit->value.integer) == GFC_INTERNAL_UNIT)))
     {
-      gfc_error ("UNIT number in INQUIRE statement at %L can not be -1", &loc);
+      gfc_error ("UNIT number in INQUIRE statement at %L cannot "
+                "be %d", &loc, (int) mpz_get_si (inquire->unit->value.integer));
       goto cleanup;
     }
 
@@ -4201,6 +4764,8 @@ gfc_resolve_inquire (gfc_inquire *inquire)
   INQUIRE_RESOLVE_TAG (&tag_size, inquire->size);
   INQUIRE_RESOLVE_TAG (&tag_s_decimal, inquire->decimal);
   INQUIRE_RESOLVE_TAG (&tag_s_iqstream, inquire->iqstream);
+  INQUIRE_RESOLVE_TAG (&tag_v_share, inquire->share);
+  INQUIRE_RESOLVE_TAG (&tag_v_cc, inquire->cc);
 #undef INQUIRE_RESOLVE_TAG
 
   if (!gfc_reference_st_label (inquire->err, ST_LABEL_TARGET))
@@ -4252,11 +4817,9 @@ match_wait_element (gfc_wait *wait)
 
   m = match_etag (&tag_unit, &wait->unit);
   RETM m = match_ltag (&tag_err, &wait->err);
-  RETM m = match_ltag (&tag_end, &wait->eor);
-  RETM m = match_ltag (&tag_eor, &wait->end);
+  RETM m = match_ltag (&tag_end, &wait->end);
+  RETM m = match_ltag (&tag_eor, &wait->eor);
   RETM m = match_etag (&tag_iomsg, &wait->iomsg);
-  if (m == MATCH_YES && !check_char_variable (wait->iomsg))
-    return MATCH_ERROR;
   RETM m = match_out_tag (&tag_iostat, &wait->iostat);
   RETM m = match_etag (&tag_id, &wait->id);
   RETM return MATCH_NO;