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

[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  89c8afe012920fe9592a510a8beab877bed1656b (commit)
       via  b66e1ec280f1f77dd9caf8077517b4be3ff35627 (commit)
       via  c08cb60fe34b42b0a83bc88b57fd41b07793863f (commit)
       via  d601805e1d9c40d88107eb4b79c6cd592d660eb2 (commit)
       via  a399b4c6a300067d6e1da58d2092129494f95711 (commit)
       via  9c8bc93a8babf7f1316d7758b8cd73f28e794f9a (commit)
      from  1ae25aae44c548a4013866d1d94476160e7dd983 (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 89c8afe012920fe9592a510a8beab877bed1656b
Author: Ralph Corderoy <address@hidden>
Date:   Mon Sep 4 10:42:17 2017 +0100

    oauth: Replace `if...return false; return true' with `return ...'.
    
    In functions returning bool, these are equivalent.
    
        Before:                 After:
            if (!foo())             return foo();
                return false;
            return true;

commit b66e1ec280f1f77dd9caf8077517b4be3ff35627
Author: Ralph Corderoy <address@hidden>
Date:   Mon Sep 4 10:29:21 2017 +0100

    oauth.c: Remove unused assignments of function return value.
    
    `if ((status = foo()) != OK) return false;' doesn't need to set local
    variable status.

commit c08cb60fe34b42b0a83bc88b57fd41b07793863f
Author: Ralph Corderoy <address@hidden>
Date:   Mon Sep 4 10:07:35 2017 +0100

    Replace FALSE and TRUE with C99's false and true.
    
    Remove the duplicate definitions of FALSE and TRUE.  Change a few
    variables from int to bool at the same time.  The semantics of bool are
    different to int because `bool b = 42' maps 42 to 1, but if the code was
    relying on that then it needs shaking out anyway.

commit d601805e1d9c40d88107eb4b79c6cd592d660eb2
Author: Ralph Corderoy <address@hidden>
Date:   Mon Sep 4 10:03:36 2017 +0100

    print_intro(): Change parameter brief from int to bool.

commit a399b4c6a300067d6e1da58d2092129494f95711
Author: Ralph Corderoy <address@hidden>
Date:   Sun Sep 3 23:37:52 2017 +0100

    flist.c: Use bool instead of int for some of the flags.

commit 9c8bc93a8babf7f1316d7758b8cd73f28e794f9a
Author: Ralph Corderoy <address@hidden>
Date:   Sun Sep 3 23:29:36 2017 +0100

    Replace boolean with bool everywhere.
    
    boolean's comment said it existed to ensure storage was a char and not
    an int so it could be packed in a struct, but the only struct using it
    doesn't care about the space taken.

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

Summary of changes:
 h/crawl_folders.h |   4 +-
 h/mf.h            |   8 ----
 h/mh.h            |   8 ----
 h/oauth.h         |  16 +++----
 h/prototypes.h    |   2 +-
 mts/smtp/smtp.c   |  23 +++++-----
 sbr/m_getfld.c    |   2 +-
 sbr/oauth.c       | 131 ++++++++++++++++++++++++++----------------------------
 sbr/oauth_prof.c  |  14 +++---
 sbr/print_help.c  |   4 +-
 uip/flist.c       |  30 ++++++-------
 uip/folder.c      |  12 ++---
 uip/install-mh.c  |   2 +-
 uip/new.c         |  12 ++---
 14 files changed, 121 insertions(+), 147 deletions(-)


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