[gawk-diffs] [SCM] gawk branch, feature/dev-5.0, updated. gawk-4.1.0-2823-ga6ebb56

[Arnold Robbins]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gawk".

The branch, feature/dev-5.0 has been updated
       via  a6ebb5697f02034ca5865940f43e016a730570df (commit)
       via  8140d635ccee51089aad9a04e9e128e5373377f7 (commit)
       via  880974d61412e6d78ee014d7010486fba2e3e1be (commit)
      from  9b85ee5535a0fd1f21b1cb43d4f5c91ef719684d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.sv.gnu.org/cgit/gawk.git/commit/?id=a6ebb5697f02034ca5865940f43e016a730570df

commit a6ebb5697f02034ca5865940f43e016a730570df
Merge: 880974d 8140d63
Author: Arnold D. Robbins <address@hidden>
Date:   Sat Oct 14 21:57:15 2017 +0300

    Merge branch 'master' into feature/dev-5.0

diff --cc ChangeLog
index 7906bed,8d5bd15..1732f02
--- a/ChangeLog
+++ b/ChangeLog
@@@ -1,22 -1,8 +1,27 @@@
  2017-10-14         Arnold D. Robbins     <address@hidden>
  
+       * field.c (do_split): Simplify the lint warnings.
+       Based on suggested code by Eric Pruitt <address@hidden>.
+ 
++      Unrelated:
++
 +      * awkgram.y (check_funcs): Remove the REALLYMEAN ifdef and
 +      simplify the lint checking code for function defined but not
 +      called or called but not defined.
 +
 +2017-10-13         Arnold D. Robbins     <address@hidden>
 +
 +      Assume a more C99 environment:
 +
 +      * awk.h: Assume we have limits.h, stdarg.h and stdbool.h.
 +      * configure.ac: Remove checks for limits.h and stdarg.h.
 +
 +2017-10-10         Arnold D. Robbins     <address@hidden>
 +
 +      * configure.ac: Remove --with-whiny-user-strftime option.
 +      * NEWS: Updated.
 +      * ChangeLog.0: Fix a typo. :-)
 +
  2017-10-08         Arnold D. Robbins     <address@hidden>
  
        * command.y: Fix the FSF's address.

http://git.sv.gnu.org/cgit/gawk.git/commit/?id=880974d61412e6d78ee014d7010486fba2e3e1be

commit 880974d61412e6d78ee014d7010486fba2e3e1be
Author: Arnold D. Robbins <address@hidden>
Date:   Sat Oct 14 21:16:39 2017 +0300

    Simply lint checking for functions defined/called.

diff --git a/ChangeLog b/ChangeLog
index a7dee7d..7906bed 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2017-10-14         Arnold D. Robbins     <address@hidden>
+
+       * awkgram.y (check_funcs): Remove the REALLYMEAN ifdef and
+       simplify the lint checking code for function defined but not
+       called or called but not defined.
+
 2017-10-13         Arnold D. Robbins     <address@hidden>
 
        Assume a more C99 environment:
diff --git a/awkgram.c b/awkgram.c
index 0c52841..aee0e2f 100644
--- a/awkgram.c
+++ b/awkgram.c
@@ -7357,22 +7357,19 @@ check_funcs()
 
        for (i = 0; i < HASHSIZE; i++) {
                for (fp = ftable[i]; fp != NULL; fp = fp->next) {
-#ifdef REALLYMEAN
-                       /* making this the default breaks old code. sigh. */
-                       if (fp->defined == 0 && ! fp->extension) {
-                               error(
-               _("function `%s' called but never defined"), fp->name);
-                               errcount++;
-                       }
-#else
-                       if (do_lint && fp->defined == 0 && ! fp->extension)
-                               lintwarn(
-               _("function `%s' called but never defined"), fp->name);
-#endif
+                       if (do_lint && ! fp->extension) {
+                               /*
+                                * Making this not a lint check and
+                                * incrementing * errcount breaks old code.
+                                * Sigh.
+                                */
+                               if (fp->defined == 0)
+                                       lintwarn(_("function `%s' called but 
never defined"),
+                                               fp->name);
 
-                       if (do_lint && fp->used == 0 && ! fp->extension) {
-                               lintwarn(_("function `%s' defined but never 
called directly"),
-                                       fp->name);
+                               if (fp->used == 0)
+                                       lintwarn(_("function `%s' defined but 
never called directly"),
+                                               fp->name);
                        }
                }
        }
diff --git a/awkgram.y b/awkgram.y
index f8b3035..178c307 100644
--- a/awkgram.y
+++ b/awkgram.y
@@ -4937,22 +4937,19 @@ check_funcs()
 
        for (i = 0; i < HASHSIZE; i++) {
                for (fp = ftable[i]; fp != NULL; fp = fp->next) {
-#ifdef REALLYMEAN
-                       /* making this the default breaks old code. sigh. */
-                       if (fp->defined == 0 && ! fp->extension) {
-                               error(
-               _("function `%s' called but never defined"), fp->name);
-                               errcount++;
-                       }
-#else
-                       if (do_lint && fp->defined == 0 && ! fp->extension)
-                               lintwarn(
-               _("function `%s' called but never defined"), fp->name);
-#endif
+                       if (do_lint && ! fp->extension) {
+                               /*
+                                * Making this not a lint check and
+                                * incrementing * errcount breaks old code.
+                                * Sigh.
+                                */
+                               if (fp->defined == 0)
+                                       lintwarn(_("function `%s' called but 
never defined"),
+                                               fp->name);
 
-                       if (do_lint && fp->used == 0 && ! fp->extension) {
-                               lintwarn(_("function `%s' defined but never 
called directly"),
-                                       fp->name);
+                               if (fp->used == 0)
+                                       lintwarn(_("function `%s' defined but 
never called directly"),
+                                               fp->name);
                        }
                }
        }

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog | 11 +++++++++++
 awkgram.c | 27 ++++++++++++---------------
 awkgram.y | 27 ++++++++++++---------------
 field.c   | 10 +++-------
 4 files changed, 38 insertions(+), 37 deletions(-)


hooks/post-receive
-- 
gawk