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

[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  95b3a95e742da57976a1cd8f723cc0819cbba53c (commit)
       via  4124cf3f320630064acf369f909b1fc13f47d212 (commit)
       via  99817f2e4d0ceaf8d3be1410c0c29fcd070ac6ec (commit)
       via  9d325068a8deee98493dd44eb62a3b66b4951462 (commit)
       via  3b6170b27ba8fedbb4d67d2842bfaa38d4417962 (commit)
       via  ea1fd31a3d41352b024fb35a1a781dce6a2f7afe (commit)
       via  877ea64f087e0327e1b63f23d29ac13fe4ad6fbd (commit)
       via  2e150215834904311e167edb9eefac4aee023fc5 (commit)
       via  ab41b15145fe5d5155fc26b1300e648a0f12d290 (commit)
       via  3e90f7c100988b1f90706e2212fec687d2cd7f52 (commit)
       via  d4ca313d4340d0ef56a9e7ce564d94f3bc09e303 (commit)
      from  a284ba0d7830603e947e44db540497b01e6b7595 (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 95b3a95e742da57976a1cd8f723cc0819cbba53c
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 23:27:00 2017 +0100

    datetime.c: Flip if-conditions so returns move from else block.
    
    Then there's no need for else-blocks after return.

commit 4124cf3f320630064acf369f909b1fc13f47d212
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 22:56:45 2017 +0100

    icalendar.l: Refer to base64.h relative to root of source.
    
    Ken reported a missing-include-file problem when building so the product
    ends up outside the source tree, e.g. running configure from elsewhere.
    It must also depend on his environment, e.g. OS or compiler, because the
    problem isn't reproducible with Arch Linux and gcc, and be specific to C
    source built from other source files, e.g. lex(1).  However, given that
    the existing sibling icalparse.h is referred to as sbr/..., duplicate
    that so it builds for Ken too.

commit 99817f2e4d0ceaf8d3be1410c0c29fcd070ac6ec
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 22:45:54 2017 +0100

    test/getcanon.c: exit(3) with 0 or 1, not -1 or symbolic error code.
    
    gethostname(3) returns -1 on failure and that was being returned from
    main;  use 1 instead.  getaddrinfo(3) returns EAI_ADDRFAMILY et al on
    failure, but here they're defined as small negative integers, unsuitable
    for main()'s return value;  again, use 1 instead.  Along the way, flip
    the logic so the end is nigh when possible, avoiding the cognitive
    overhead of tracking state when continuing to read through the source.

commit 9d325068a8deee98493dd44eb62a3b66b4951462
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 22:08:43 2017 +0100

    test/getfullname.c: Move return case first in if-else chain.

commit 3b6170b27ba8fedbb4d67d2842bfaa38d4417962
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 21:56:42 2017 +0100

    Refer to #include files from the root of nmh's source.
    
    One of the -I options given to the C compiler is the root of the nmh
    source.  This means uip/foo.c's #include of the relative ../sbr/bar.h
    can also be written more tidily as sbr/bar.h.

commit ea1fd31a3d41352b024fb35a1a781dce6a2f7afe
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 21:30:39 2017 +0100

    icalparse.y: Remove else-block that returns by merging if-conditions.
    
    The code tested two values to see if they were present, and if they
    were, and were equal, then it returned this node.  If either were
    missing then this node was still returned because it matched the more
    limited criteria that had already been tested.  So the test for
    returning this node can more simply be if either value to compare is
    missing, or they're (both present and) equal.

commit 877ea64f087e0327e1b63f23d29ac13fe4ad6fbd
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 18:29:58 2017 +0100

    seq_setprev.c: Flip if-condition so return moves from else block.
    
    Then no need for else block after return.

commit 2e150215834904311e167edb9eefac4aee023fc5
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 18:28:31 2017 +0100

    folder_pack.c: Flip if-condition so return moves from else block.
    
    Then no need for else block after return.

commit ab41b15145fe5d5155fc26b1300e648a0f12d290
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 18:27:42 2017 +0100

    seq_setunseen.c: Flip if-condition so return moves from else block.
    
    Then no need for else block after return.

commit 3e90f7c100988b1f90706e2212fec687d2cd7f52
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 18:26:45 2017 +0100

    mhstoresbr.c: Flip if-condition so return moves from else block.
    
    Then no need for else block after return.

commit d4ca313d4340d0ef56a9e7ce564d94f3bc09e303
Author: Ralph Corderoy <address@hidden>
Date:   Sat Sep 9 18:20:09 2017 +0100

    read_switch_multiword.c: Flip if-condition so return moves from else block.
    
    Then no need for else block after return.

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

Summary of changes:
 config/config.c             |  2 +-
 mts/smtp/smtp.c             |  2 +-
 sbr/datetime.c              | 52 ++++++++++++++++++++-------------------------
 sbr/folder_pack.c           |  5 ++---
 sbr/icalendar.l             |  2 +-
 sbr/icalparse.y             |  7 +-----
 sbr/read_switch_multiword.c |  5 ++---
 sbr/seq_setprev.c           | 14 ++++++------
 sbr/seq_setunseen.c         | 14 ++++++------
 test/getcanon.c             | 45 +++++++++++++++++++--------------------
 test/getfullname.c          | 11 +++++-----
 uip/anno.c                  |  2 +-
 uip/annosbr.c               |  4 ++--
 uip/ap.c                    |  2 +-
 uip/burst.c                 |  4 ++--
 uip/comp.c                  |  2 +-
 uip/dist.c                  |  2 +-
 uip/distsbr.c               |  2 +-
 uip/dp.c                    |  2 +-
 uip/dropsbr.c               |  4 ++--
 uip/flist.c                 |  2 +-
 uip/fmttest.c               |  4 ++--
 uip/folder.c                |  2 +-
 uip/forw.c                  |  2 +-
 uip/forwsbr.c               |  2 +-
 uip/inc.c                   |  6 +++---
 uip/install-mh.c            |  6 +++---
 uip/mark.c                  |  2 +-
 uip/mhbuild.c               |  4 ++--
 uip/mhbuildsbr.c            |  6 +++---
 uip/mhcachesbr.c            |  4 ++--
 uip/mhfixmsg.c              |  6 +++---
 uip/mhfree.c                |  2 +-
 uip/mhlist.c                |  2 +-
 uip/mhlogin.c               |  2 +-
 uip/mhlsbr.c                |  4 ++--
 uip/mhmisc.c                |  2 +-
 uip/mhn.c                   |  2 +-
 uip/mhoutsbr.c              |  2 +-
 uip/mhparse.c               |  4 ++--
 uip/mhpath.c                |  2 +-
 uip/mhshow.c                |  4 ++--
 uip/mhshowsbr.c             |  2 +-
 uip/mhstore.c               |  2 +-
 uip/mhstoresbr.c            | 11 +++++-----
 uip/new.c                   |  4 ++--
 uip/packf.c                 |  2 +-
 uip/pick.c                  |  2 +-
 uip/popsbr.c                |  2 +-
 uip/post.c                  |  4 ++--
 uip/prompter.c              |  2 +-
 uip/rcvdist.c               |  2 +-
 uip/rcvstore.c              |  6 +++---
 uip/rcvtty.c                |  2 +-
 uip/refile.c                |  4 ++--
 uip/repl.c                  |  2 +-
 uip/rmf.c                   |  4 ++--
 uip/rmm.c                   |  2 +-
 uip/scan.c                  |  4 ++--
 uip/scansbr.c               |  2 +-
 uip/send.c                  |  4 ++--
 uip/sendsbr.c               |  6 +++---
 uip/show.c                  |  2 +-
 uip/slocal.c                |  4 ++--
 uip/sortm.c                 |  2 +-
 uip/viamail.c               |  4 ++--
 uip/whatnowsbr.c            |  6 +++---
 uip/whom.c                  |  4 ++--
 68 files changed, 163 insertions(+), 183 deletions(-)


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