[gawk-diffs] [SCM] gawk branch, master, updated. gawk-4.1.0-2724-g4e11acf

[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, master has been updated
       via  4e11acf5f6be5c228aa2728e51cb9f17874a599d (commit)
       via  e2efe866b0fe0b697f633612893da5e74034a47c (commit)
      from  833b375c19c372f13648e4f88f4177dad7350c5e (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=4e11acf5f6be5c228aa2728e51cb9f17874a599d

commit 4e11acf5f6be5c228aa2728e51cb9f17874a599d
Author: Arnold D. Robbins <address@hidden>
Date:   Thu Aug 17 20:35:30 2017 +0300

    Bump version in configure.ac.

diff --git a/configure b/configure
index 50c92f8..12e51da 100755
--- a/configure
+++ b/configure
@@ -1,6 +1,6 @@
 #! /bin/sh
 # Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.69 for GNU Awk 4.1.62.
+# Generated by GNU Autoconf 2.69 for GNU Awk 4.1.63.
 #
 # Report bugs to <address@hidden>.
 #
@@ -580,8 +580,8 @@ MAKEFLAGS=
 # Identity of this package.
 PACKAGE_NAME='GNU Awk'
 PACKAGE_TARNAME='gawk'
-PACKAGE_VERSION='4.1.62'
-PACKAGE_STRING='GNU Awk 4.1.62'
+PACKAGE_VERSION='4.1.63'
+PACKAGE_STRING='GNU Awk 4.1.63'
 PACKAGE_BUGREPORT='address@hidden'
 PACKAGE_URL='http://www.gnu.org/software/gawk/'
 
@@ -1331,7 +1331,7 @@ if test "$ac_init_help" = "long"; then
   # Omit some internal or obsolete options to make the list less imposing.
   # This message is too long to be a string in the A/UX 3.1 sh.
   cat <<_ACEOF
-\`configure' configures GNU Awk 4.1.62 to adapt to many kinds of systems.
+\`configure' configures GNU Awk 4.1.63 to adapt to many kinds of systems.
 
 Usage: $0 [OPTION]... [VAR=VALUE]...
 
@@ -1401,7 +1401,7 @@ fi
 
 if test -n "$ac_init_help"; then
   case $ac_init_help in
-     short | recursive ) echo "Configuration of GNU Awk 4.1.62:";;
+     short | recursive ) echo "Configuration of GNU Awk 4.1.63:";;
    esac
   cat <<\_ACEOF
 
@@ -1526,7 +1526,7 @@ fi
 test -n "$ac_init_help" && exit $ac_status
 if $ac_init_version; then
   cat <<\_ACEOF
-GNU Awk configure 4.1.62
+GNU Awk configure 4.1.63
 generated by GNU Autoconf 2.69
 
 Copyright (C) 2012 Free Software Foundation, Inc.
@@ -2235,7 +2235,7 @@ cat >config.log <<_ACEOF
 This file contains any messages produced by compilers while
 running configure, to aid debugging if configure makes a mistake.
 
-It was created by GNU Awk $as_me 4.1.62, which was
+It was created by GNU Awk $as_me 4.1.63, which was
 generated by GNU Autoconf 2.69.  Invocation command line was
 
   $ $0 $@
@@ -3118,7 +3118,7 @@ fi
 
 # Define the identity of the package.
  PACKAGE='gawk'
- VERSION='4.1.62'
+ VERSION='4.1.63'
 
 
 cat >>confdefs.h <<_ACEOF
@@ -11562,7 +11562,7 @@ cat >>$CONFIG_STATUS <<\_ACEOF || ac_write_fail=1
 # report actual input values of CONFIG_FILES etc. instead of their
 # values after options handling.
 ac_log="
-This file was extended by GNU Awk $as_me 4.1.62, which was
+This file was extended by GNU Awk $as_me 4.1.63, which was
 generated by GNU Autoconf 2.69.  Invocation command line was
 
   CONFIG_FILES    = $CONFIG_FILES
@@ -11630,7 +11630,7 @@ _ACEOF
 cat >>$CONFIG_STATUS <<_ACEOF || ac_write_fail=1
 ac_cs_config="`$as_echo "$ac_configure_args" | sed 's/^ //; 
s/[\\""\`\$]/\\\\&/g'`"
 ac_cs_version="\\
-GNU Awk config.status 4.1.62
+GNU Awk config.status 4.1.63
 configured by $0, generated by GNU Autoconf 2.69,
   with options \\"\$ac_cs_config\\"
 
diff --git a/configure.ac b/configure.ac
index 872874b..1a1fdac 100644
--- a/configure.ac
+++ b/configure.ac
@@ -23,7 +23,7 @@ dnl
 
 dnl Process this file with autoconf to produce a configure script.
 
-AC_INIT([GNU Awk],[4.1.62],address@hidden,[gawk])
+AC_INIT([GNU Awk],[4.1.63],address@hidden,[gawk])
 
 # This is a hack. Different versions of install on different systems
 # are just too different. Chuck it and use install-sh.

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

commit e2efe866b0fe0b697f633612893da5e74034a47c
Author: Arnold D. Robbins <address@hidden>
Date:   Thu Aug 17 20:33:20 2017 +0300

    Fix reference counting management problem in rebuild_record().

diff --git a/ChangeLog b/ChangeLog
index 6741478..aceeac3 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2017-08-17         Arnold D. Robbins     <address@hidden>
+
+       * field.c (rebuild_record): Set new fields valref to 1 if
+       original field's valref was > 1. Update the comment. Found
+       by running chem.
+
 2017-08-16         Arnold D. Robbins     <address@hidden>
 
        * gawkapi.c (assign_number): Clean up the code a bit.
diff --git a/field.c b/field.c
index 3e29ec2..0685e9b 100644
--- a/field.c
+++ b/field.c
@@ -217,20 +217,16 @@ rebuild_record()
                        *n = *r;
                        if (r->valref > 1) {
                                /*
-                                * This probably never happens, since it
-                                * was not considered by previous versions of
-                                * this function. But it seems clear that
+                                * This can and does happen.  It seems clear 
that
                                 * we can't leave r's stptr pointing into the
                                 * old $0 buffer that we are about to unref.
-                                * It's not a priori obvious that valref must be
-                                * 1 in all cases, so it seems wise to suppport
-                                * this corner case. The only question is
-                                * whether to add a warning message.
                                 */
                                emalloc(r->stptr, char *, r->stlen + 1, 
"rebuild_record");
                                memcpy(r->stptr, cops, r->stlen);
                                r->stptr[r->stlen] = '\0';
                                r->flags |= MALLOC;
+
+                               n->valref = 1;  // reset in the new field to 
start it off correctly!
                        }
 
                        n->stptr = cops;

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

Summary of changes:
 ChangeLog    |  6 ++++++
 configure    | 20 ++++++++++----------
 configure.ac |  2 +-
 field.c      | 10 +++-------
 4 files changed, 20 insertions(+), 18 deletions(-)


hooks/post-receive
-- 
gawk