[gawk-diffs] [SCM] gawk branch, feature/fixtype, updated. gawk-4.1.0-1878-ga4cbdf3

[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/fixtype has been updated
       via  a4cbdf37b98a56f688f8e1be96666a6f109c0e38 (commit)
       via  e8a8d49954bb50b2b67a88382e80fe63e738cd3f (commit)
       via  82024c4ac22e50de3b0ad36d8aa81c52e7b4c9ae (commit)
       via  bc1cf4039d9e52ae7b285f20e68a846d9143a210 (commit)
       via  5da46e952e81f827abc4b75bd79178cfc2875460 (commit)
       via  783b841df282ebb80a8f4bc90d668d42ebb7bbd1 (commit)
       via  93897ef9a4907b5969a5496501c07ecb537dbbc8 (commit)
      from  96d37ecb22d847499fbfab80c62894b64249b8c4 (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=a4cbdf37b98a56f688f8e1be96666a6f109c0e38

commit a4cbdf37b98a56f688f8e1be96666a6f109c0e38
Merge: 96d37ec e8a8d49
Author: Arnold D. Robbins <address@hidden>
Date:   Fri Jul 1 07:49:30 2016 +0300

    Merge branch 'master' into feature/fixtype

diff --cc ChangeLog
index 0caafa6,ac7aa53..c125050
--- a/ChangeLog
+++ b/ChangeLog
@@@ -1,83 -1,16 +1,96 @@@
+ 2016-07-01         Arnold D. Robbins     <address@hidden>
+ 
+       * array.c (value_info): Print something reasonable when stfmt
+       is -1.
+       * mpfr.c (mpg_format_val): Don't cast index to char.
+       * node.c (r_format_val): Ditto.
+       Thanks to Andrew Schorr for pointing these out.
+ 
+       Unrelated:
+ 
+       * gawkapi.c (api_warning): Fix the comment header.
+       (api_lintwarn): Factor out the call to va_end to after the if.
+ 
 +2016-06-30         Arnold D. Robbins     <address@hidden>
 +
 +      * node.c (r_force_number): Coding style change.
 +
 +2016-06-30         Andrew J. Schorr     <address@hidden>
 +
 +      * awk.h (STFMT_UNUSED): New define indicating that the string
 +      representation does not depend on CONVFMT or OFMT.
 +      (force_string): Use STFMT_UNUSED to improve code clarity.
 +      * array.c (value_info): Fix stfmt logic.
 +      * builtin.c (do_print): Use STFMT_UNUSED to improve code clarity.
 +      * field.c (set_record): Ditto.
 +      * gawkapi.c (api_sym_update_scalar): Ditto.
 +      * int_array.c (is_integer): Check stfmt equals STFMT_UNUSED before
 +      bothering to inspect the string.
 +      * mpfr.c (mpg_format_val): Use STFMT_UNUSED to improve code clarity.
 +      Remove buggy cast to char in stfmt assignment.
 +      * node.c (r_format_val): Ditto.
 +      * str_array.c (str_lookup): Use STFMT_UNUSED to improve code clarity.
 +      * symbol.c (check_param_names): Ditto.
 +
 +2016-06-29         Andrew J. Schorr     <address@hidden>
 +
 +      * node.c (r_force_number): Optimize by trimming leading and trailing
 +      white space before we inspect the string contents.
 +      (get_ieee_magic_val): Must terminate the string with '\0' before
 +      calling strtod.
 +
 +2016-06-27         Andrew J. Schorr     <address@hidden>
 +
 +      * gawkapi.h (awk_string): Add comment about the potential lack of
 +      NUL-termination.
 +
 +2016-06-27         Andrew J. Schorr     <address@hidden>
 +
 +      * awk.h: Add a comment regarding the potential lack of NUL-termination
 +      for Node_val strings.
 +
 +2016-06-27         Andrew J. Schorr     <address@hidden>
 +
 +      * node.c (r_format_val): Do not free stptr unless STRCUR is set.
 +      This is safer than testing for non-NULL stptr, since, for example,
 +      pp_number copies a node and calls r_format_val, but does not bother
 +      to set stptr to NULL beforehand.
 +
 +2016-06-26         Andrew J. Schorr     <address@hidden>
 +
 +      * node.c (r_force_number): When checking for trailing spaces, protect
 +      against running off the end of the string.
 +      * mpfr.c (force_mpnum): Ditto.
 +
 +2016-06-26         Andrew J. Schorr     <address@hidden>
 +
 +      * builtin.c (do_print): There's actually no reason to test whether a
 +      value is a number, since the STRCUR flag and stfmt value contain all
 +      the necessary info, as in awk.h:force_string.
 +
 +2016-06-26         Andrew J. Schorr     <address@hidden>
 +
 +      * builtin.c (do_print): Do not use OFMT to print strnum values. We
 +      accomplish this by calling format_val for a NUMBER only
 +      if there is no string currently available, or if stfmt equals
 +      neither -1 nor OFMTidx.
 +
 +2016-06-26         Arnold D. Robbins     <address@hidden>
 +
 +      * awk.h: Edit some comments. Add others. Minor coding style changes.
 +      * builtin.c (format_tree): Restore a comment.
 +      (do_mktime): Restore saving/restoring of byte after format string.
 +      (do_sub): Coding style. Use %.*s in warning message.
 +      (nondec2awknum): Restore saving/restoring of byte after string value
 +      being converted.
 +      * eval.c: Minor coding style edits.
 +      * int_array.c (is_integer): Fix order of checks for not
 +      updating string value: check length == 0 before testing values.
 +      Coding style edits.
 +      * mpfr.c (do_mpfr_strtonum): Coding style edits.
 +      * node.c (r_force_number): Restore saving/restoring of byte after
 +      string value being converted. Edit comments some.
 +
  2016-06-26         Arnold D. Robbins     <address@hidden>
  
        Repair change of 2015-08-25 to handling of MAYBE_NUM.
diff --cc array.c
index ddd3c08,edc192c..d9c80a3
--- a/array.c
+++ b/array.c
@@@ -704,14 -704,7 +704,14 @@@ value_info(NODE *n
        if ((n->flags & (STRING|STRCUR)) == STRCUR) {
                fprintf(output_fp, "][");
                fprintf(output_fp, "stfmt=%d, ", n->stfmt);     
 -              fprintf(output_fp, "CONVFMT=\"%s\"", n->stfmt <= -1 ? "<unused>"
 +              /*
 +               * If not STFMT_UNUSED, could be CONVFMT or OFMT if last
 +               * used in a print statement. If immutable, could be that it
 +               * was originally set as a string, or it's a number that has
 +               * an integer value.
 +               */
 +              fprintf(output_fp, "FMT=\"%s\"",
-                                       n->stfmt == STFMT_UNUSED ? "%s"
++                                      n->stfmt == STFMT_UNUSED ? "<unused>"
                                        : fmt_list[n->stfmt]->stptr);
        }
  
diff --cc test/ChangeLog
index cca91c5,57b23ab..f0f346e
--- a/test/ChangeLog
+++ b/test/ChangeLog
@@@ -1,18 -1,8 +1,23 @@@
+ 2016-07-01         Arnold D. Robbins     <address@hidden>
+ 
+       * arrayind1.awk, arrayind1.ok: Comment out prints to stderr to
+       avoid buffer flushing on obscure systems.
+ 
 +2016-06-26         Andrew J. Schorr     <address@hidden>
 +
 +      * strnum2.ok: Fix results, since print for a strnum should not be
 +      affected by OFMT or CONVFMT.
 +
 +2016-06-22         Andrew J. Schorr     <address@hidden>
 +
 +      * strnum2.awk, strnum2.ok: Improve test case to show both OFMT and
 +      CONVFMT conversions.
 +
 +2016-06-20         Andrew J. Schorr     <address@hidden>
 +
 +      * Makefile.am (strnum2): New test.
 +      * strnum2.awk, strnum2.ok: New files.
 +
  2016-06-14         Arnold D. Robbins     <address@hidden>
  
        * Makefile.am (subback): New test.

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

Summary of changes:
 ChangeLog             |   13 +++++++++++++
 array.c               |    2 +-
 extension/ChangeLog   |    9 +++++++++
 extension/inplace.c   |    1 +
 extension/revtwoway.c |    8 +++++---
 gawkapi.c             |    3 +--
 pc/ChangeLog          |    4 ++++
 pc/Makefile.tst       |   15 ++++++++++++---
 test/ChangeLog        |    5 +++++
 test/arrayind1.awk    |   10 +++++-----
 test/arrayind1.ok     |    5 -----
 11 files changed, 56 insertions(+), 19 deletions(-)


hooks/post-receive
-- 
gawk