[Nmh-commits] [SCM] The nmh Mail Handling System branch, master, updated. 1.6-branchpoint-924-g5b01163

[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  5b01163dec51496cc4c50648d6674d67f0092a33 (commit)
       via  1e424a2249aa6b911fd3be973de00cee413342eb (commit)
       via  989be1fb92afdec3842ac931fdcbced29ab0ba39 (commit)
       via  f0b7d9a7c3b81f3cf3eb9715da1003770f1a1086 (commit)
      from  92884625549fcd1e49acf8fde1d54bd537a80072 (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 5b01163dec51496cc4c50648d6674d67f0092a33
Author: Ralph Corderoy <address@hidden>
Date:   Sat Oct 22 15:53:51 2016 +0100

    Check advertise()'s iob[] isn't too short with assert(3).
    
    Using DIM(a) added to h/mh.h

commit 1e424a2249aa6b911fd3be973de00cee413342eb
Author: Ralph Corderoy <address@hidden>
Date:   Sat Oct 22 15:36:40 2016 +0100

    Use macros in advertise() to simplify source.
    
    Happens to remove strlen() of literals at the same time.

commit 989be1fb92afdec3842ac931fdcbced29ab0ba39
Author: Ralph Corderoy <address@hidden>
Date:   Sat Oct 22 14:43:11 2016 +0100

    Document advise(), adios(), admonish(), and advertise().
    
    It doesn't make their design any better.  Their "ad" prefix goes against
    common advice of having distinguishable identifiers to avoid mix-ups,
    and seems to be a theme that got out of hand.  The out-of-order
    parameters to advertise seem due to lack of strerror(3) that's no longer
    a problem.
    
    Before:
        advertise("open", "continuing...", "couldn't read profile: %s",
            path);
    Could be:
        error("couldn't read profile: %s: open: %s, continuing...\n",
            path, strerror(errno));
    And the "open" is typically dropped as superfluous so there's a
    repetitive leading NULL passed a lot.

commit f0b7d9a7c3b81f3cf3eb9715da1003770f1a1086
Author: Ralph Corderoy <address@hidden>
Date:   Sat Oct 22 13:58:43 2016 +0100

    POSIX, 2016 Ed, removes bcopy(3).  Stop referring to it.
    
    Even though it's in a comment, it shows up in searches.

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

Summary of changes:
 h/mh.h         |    3 ++
 sbr/error.c    |   94 ++++++++++++++++++++++++++++++++------------------------
 sbr/m_getfld.c |    4 +--
 3 files changed, 58 insertions(+), 43 deletions(-)


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