[Nmh-commits] [SCM] The nmh Mail Handling System branch, master, updated. 1.7-branchpoint-27-g0ff76f3

[Ralph Corderoy]

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 "The nmh Mail Handling System".

The branch, master has been updated
       via  0ff76f3b2823f98e75b76dc93f2b5be293171610 (commit)
       via  4165ce30f81bc078f3635a2a198f3a2ab5fc9b75 (commit)
       via  4ff236a53b9caf73f0583c41e8be49a77546dd20 (commit)
       via  210b50a5ac585ae99db277fdae5638772df9c3d9 (commit)
      from  9c7b5289d70448fb5b118abbea5a4d37afdf00be (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 -----------------------------------------------------------------
commit 0ff76f3b2823f98e75b76dc93f2b5be293171610
Author: Ralph Corderoy <address@hidden>
Date:   Thu Aug 10 15:44:20 2017 +0100

    m_getfld() et al: Add wrapper without FILE *iob parameter.
    
    m_getfld(), m_getfld_track_filepos(), and m_unknown() all take a FILE
    *iob that's used to update *m_getfld_state_t, even though it's probably,
    hopefully, the same as the previous calls.  Create wrapper functions for
    these with an arbitrary `2' suffix that don't have this parameter.
    After checking the m_getfld_state_t is initialised, they pass in its
    existing FILE *iob.  This allows callers to transition to the wrappers
    in the cases where it's obviously the same FILE *iob being passed in.
    It isn't always obvious, in part because of the convention for calling
    this variable `in', even when it's a file-level static, a word that also
    occurs in comments.

commit 4165ce30f81bc078f3635a2a198f3a2ab5fc9b75
Author: Ralph Corderoy <address@hidden>
Date:   Thu Aug 10 15:30:54 2017 +0100

    m_getfld.c: Make m_getfld_state_init() public.
    
    This allows callers to use it if they wish to pass in its FILE *iob.

commit 4ff236a53b9caf73f0583c41e8be49a77546dd20
Author: Ralph Corderoy <address@hidden>
Date:   Thu Aug 10 00:08:55 2017 +0100

    m_getfld.c: Alter Peek() to call Getc() and Ungetc().
    
    Functionally equivalent, not called too often, and it gets rid of
    another copy of the logic to decide if we've enough, should read some
    more, etc.

commit 210b50a5ac585ae99db277fdae5638772df9c3d9
Author: Ralph Corderoy <address@hidden>
Date:   Wed Aug 9 23:56:27 2017 +0100

    m_getfld.c: Alter Peek() to match Getc()'s structure.
    
    Their logic was equivalent, but written differently, which didn't help
    the reader easily see they were the same apart from whether the returned
    character was consumed.  Now they're visually similar too.

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

Summary of changes:
 h/prototypes.h |  5 +++++
 sbr/m_getfld.c | 60 ++++++++++++++++++++++++++++++++++++++++++++--------------
 2 files changed, 51 insertions(+), 14 deletions(-)


hooks/post-receive
-- 
The nmh Mail Handling System