Changeset 1989 for vendor/gnumake/current/read.c

Timestamp:

Oct 29, 2008, 12:02:45 AM (17 years ago)

Author:

bird

Message:

Load gnumake-2008-10-28-CVS into vendor/gnumake/current.

File:

• vendor/gnumake/current/read.c (modified) (14 diffs)

vendor/gnumake/current/read.c

@@ -1 +1 @@
 /* Reading and parsing of makefiles for GNU Make.
 Copyright (C) 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997,
-1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006 Free Software
+1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007 Free Software
 Foundation, Inc.
 This file is part of GNU Make.
@@ -7 +7 @@
 GNU Make is free software; you can redistribute it and/or modify it under the
 terms of the GNU General Public License as published by the Free Software
-Foundation; either version 2, or (at your option) any later version.
+Foundation; either version 3 of the License, or (at your option) any later
+version.
 
 GNU Make is distributed in the hope that it will be useful, but WITHOUT ANY
@@ -14 +15 @@
 
 You should have received a copy of the GNU General Public License along with
-GNU Make; see the file COPYING.  If not, write to the Free Software
-Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.  */
+this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "make.h"
@@ -468 +468 @@
 #define record_waiting_files()                                                \
   do                                                                          \
-    {                                                                         \
+    {                                                                         \
       if (filenames != 0)                                                     \
         {                                                                     \
@@ -541 +541 @@
                 continue;
 
-              /* Append this command line to the line being accumulated.  */
+              /* Append this command line to the line being accumulated.
+                 Strip command prefix chars that appear after newlines.  */
               if (commands_idx == 0)
                 cmds_started = ebuf->floc.lineno;
 
-              if (linelen + 1 + commands_idx > commands_len)
+              if (linelen + commands_idx > commands_len)
                 {
-                  commands_len = (linelen + 1 + commands_idx) * 2;
+                  commands_len = (linelen + commands_idx) * 2;
                   commands = xrealloc (commands, commands_len);
                 }
-              memcpy (&commands[commands_idx], line, linelen);
-              commands_idx += linelen;
-              commands[commands_idx++] = '\n';
+              p = &commands[commands_idx];
+              p2 = line + 1;
+              while (--linelen)
+                {
+                  ++commands_idx;
+                  *(p++) = *p2;
+                  if (p2[0] == '\n' && p2[1] == cmd_prefix)
+                    {
+                      ++p2;
+                      --linelen;
+                    }
+                  ++p2;
+                }
+              *p = '\n';
+              ++commands_idx;
 
               continue;
@@ -575 +588 @@
 
       /* Compare a word, both length and contents. */
-#define word1eq(s)      (wlen == sizeof(s)-1 && strneq (s, p, sizeof(s)-1))
+#define word1eq(s)      (wlen == sizeof(s)-1 && strneq (s, p, sizeof(s)-1))
       p = collapsed;
       while (isspace ((unsigned char)*p))
@@ -613 +626 @@
       if (!in_ignored_define)
         {
-          int i = conditional_line (p, wlen, fstart);
+          int i = conditional_line (p, wlen, fstart);
           if (i != -2)
             {
@@ -849 +862 @@
          variable definition.  But now we know it is definitely lossage.  */
       if (line[0] == cmd_prefix)
-        fatal(fstart, _("commands commence before first target"));
+        fatal(fstart, _("recipe commences before first target"));
 
       /* This line describes some target files.  This is complicated by
@@ -900 +913 @@
           case w_eol:
             if (cmdleft != 0)
-              fatal(fstart, _("missing rule before commands"));
+              fatal(fstart, _("missing rule before recipe"));
             /* This line contained something but turned out to be nothing
                but whitespace (a comment?).  */
@@ -988 +1001 @@
                  one of the most common bugs found in makefiles...  */
               fatal (fstart, _("missing separator%s"),
-                     !strneq(line, "        ", 8) ? ""
-                     : _(" (did you mean TAB instead of 8 spaces?)"));
+                     (cmd_prefix == '\t' && !strneq(line, "        ", 8))
+                     ? "" : _(" (did you mean TAB instead of 8 spaces?)"));
             continue;
           }
@@ -1881 +1894 @@
      See Savannah bug # 12124.  */
   if (snapped_deps)
-    fatal (flocp, _("prerequisites cannot be defined in command scripts"));
+    fatal (flocp, _("prerequisites cannot be defined in recipes"));
 
   if (commands_idx > 0)
@@ -1981 +1994 @@
             {
               error (&cmds->fileinfo,
-                     _("warning: overriding commands for target `%s'"),
+                     _("warning: overriding recipe for target `%s'"),
                      f->name);
               error (&f->cmds->fileinfo,
-                     _("warning: ignoring old commands for target `%s'"),
+                     _("warning: ignoring old recipe for target `%s'"),
                      f->name);
             }
@@ -2425 +2438 @@
       /* Add it to the front of the chain.  */
       new1 = xmalloc (size);
+      memset (new1, '\0', size);
       new1->name = name;
       new1->next = new;
@@ -2977 +2991 @@
           if (len > max_incl_len)
             max_incl_len = len;
-          dirs[idx++] = strcache_add_len (*cpp, len - 1);
+          dirs[idx++] = strcache_add_len (*cpp, len);
         }
     }