Port fixes from trunk (SASL fixes and mishandling of reply string buffer)

Backport fix from trunk for SASL bug.

Copy version number etc updates from trunk to 1.3 branch (they should
ideally have been done on this branch in the first place)

Use hyphen, not plus (corresponding to previous RCx naming convention)

Bump version number, date for release of 1.3 RC1

move most remaining macros out of acconfig.h which is an obsolete feature of
autoconf

* bug #18655: fix use of admonish() for a fatal error (should
be adios(); only actual effect would be wrong exit code).
Thanks to Craig Leres for spotting this.

Bug #20028 (Debian bug 399271): fix code assuming that pointer
differences were 32 bits -- thanks to Dean Gaudet for the patch.

* uip/mhshowsbr.c (show_all_messages): Be more generous when parsing
    multipart messages.

Insert types for args of some function pointers to improve type checking.

Return type of (*done)() changed to void. default_done() replaced by
exit(). All dead code "return 1" lines removed from *_done()s and replaced
by explicit "return 1" following done() calls in main()s (they should
never be reached). This should make cleaning up program termination
structures easier.

Refactor cts pointer and free()ing on abnormal exit of associated list
into mhfree.c, eliminating duplication.

Change link-time overriding of done() to run-time reassigning of equivalent
(*done)(). This should make changing the return type of done and the
rationalisation of exit code easier.

  The MH web site is now rand-mh.sourceforge.net/ and the MH
  book is now at http://rand-mh.sourceforge.net/book/.

  Changed Questions

1.02 ...current version? mailutils 1.2
1.03 Where can I get MH? Debian 4.0 nmh 1.1-RC4, update links, mu 1.2
1.04 ...references for MH? Refer to SourceForge
1.06 How...print a MH manual? Refer to SourceForge
2.17 How...build MH on a Mac? Use fink
4.05 ...tools to archive MH? Mention swish++, mairix, and namazu

* Red Hat Bug #253342: inc.c, utils.c, utils.h: When inc is run with the
-silent flag, don't exit(1) for no apparent reason.

Rename functions to avoid clash with C library exp2().

Suppress 'uninitialised variable' warning.

On Linux, define _GNU_SOURCE so that GNU libc exposes the wcwidth()
prototype in wchar.h.

Change name of internal function to avoid potential conflicts with GNU
libc's getline() function.

Minor rearrangement so variables/functions used only if !HAVE_LIBLOCKFILE
are also declared only in that case.

For gcc, make the default CFLAGS include -Wno-pointer-sign if the compiler
recognises it. gcc 4 now by default produces a large volume of these
warnings, which I believe are probably spurious and definitely a lot of
work to go through and fix. So we suppress them for now so that the other
warnings are more easily visible.

buffer is compared with cp, and so needs to be changed to unsigned too.

Initial checkin of a testsuite

Change all chars being passed to the ctype macros (isalph(), etc.) to
be unsigned chars, following through to change any other types as
necessary (e.g. comparison between two pointers). Not all chars have
been changed to unsigned chars; just those that were in danger of being
sign extended when passed to the int-taking ctype macros.

We're not using the .Bu macro anymore.

Fix for bug #11139: mhl -form mhl.reply omits '> ' on some lines.

putcomp() was assuming only the first call on the body had the buffer
beginning at the start of a line.

Application of patch #5218: patch for bug #9742. Fixes a typo in the
size arg of snprintf(). mhlist output which used to have incorrect lines
such as
     1.2   multipart/mixed            772
     1.21  text/plain                   9
     1.22  multipart/mixed            162
now have the correct
     1.2   multipart/mixed            772
     1.2.1 text/plain                   9
     1.2.2 multipart/mixed            162

bug #18630, #18631, #18632, #18634: various patches from Craig Leres
fixing error message argument problems.

bug #15212: remove configure test for broken AT&T vi. This test was broken
and unfixable. Support for manually defining ATTVIBUG remains.

Add a missing header file

Changed bold "-all" to unbolded "all". It's a reserved msgs word, not a flag.

* uip/sendsbr.c: with attachformat values of 1 or 2, add
name= portion to Content-Type header.  This makes them
consistent with attachformat value 0.  And it allows mhstore
to use that (file)name when extracting attachments.

* configure.in: If we're not using dotlocking, there is no need to set inc
sgid.

Fixed bug introduced when changing from check_folder() to new
create_folder() function. The folder name needed to be expanded to an
absolute path first. Similar things may have happened with other
conversions.

Created pluspath function to check for '+' prefix and change flag for
path() appropriately. This eliminates some duplicate prefix processing,
but not all. The path functions need to be restructed; this is the first
step.

ssequal() does not check for a substring, but for a prefix, so added a
comment to this effect. If this is sufficient, the function body should
remain unchanged but the name and comments should be modified. If not,
then the function needs to be corrected.

fixes case where a very long character-class range (128 or
more characters) causes negative indexing into pattern string.

Fixed a typo.

Created function open_form() to open form file or fallback to default
components file, with error handling. Replaced duplicated code with call
to this function.

Fixed a couple of typos.

Sanitised some dodgy homedir finding code, but it still might not make
sense.

Get rid of some trivial warnings; unused variables, ambiguous if-else,
missing headers.

Make print_aka() and print_usr() static. They're not used outside ali.c.

Created app_msgarg() (append message arg) and a simple resizable array
struct to go with it that do almost exactly what the code they've replaced
was doing.
Replaced this (duplicated) code in many files with a call to app_msgarg().

Completing the .cvsignore list(s).

Fixed a small typo.

this commit:
    - adds range support to character classes
    - fixes out-of-bounds references to the case-folding array
        when doing case comparisons for 8-bit message text
    - makes character classes as case tolerant as the rest
        of the pick regexp machine.  (where lowercase chars in
        patterns should match uppercase in the text.)
    - fixes an uninitialized pointer warning, which might actually have
        been a real bug

* uip/sendsbr.c: with attachformat values of 1 or 2, only
generate Content-Disposition headers for MIME attachments, not
for the message contents themselves.  Microsoft Outlook Build
10.0.6626, at least, doesn't show the message contents if they
have Content-Disposition.

  Changed Questions

1.03 Where can I get MH? nmh 1.2

Since faqs.org appears to be moribund, I'm now using my site as the
official location of the MH FAQ. I added some missing quotes in
various places, so now Emacs highlighting works better.

    New Questions

5.26 How do I fix a bogus In-Reply-To or missing References field?

  Changed Questions

1.02 current version? nmh 1.2
1.04 references for MH? New FAQ official site. MH-E manual URL
5.15 make sense of replcomps? Use 1.1 version

* sbr/fmt_rfc2047.c (decode_rfc2047): Don't close the iconv descriptor if
  it isn't valid.

* (mh_strcasecmp): Rename the private strcasecmp function to mh_strcasecmp.

and close iconv handle if encoded block empty

don't try to malloc 0 bytes if an RFC2047 encoded block is empty

Install scripts with INSTALL_SCRIPT so INSTALL_PROGRAM can be set to
'install -s' without it causing the installation of the scripts to fail.

* uip/sendsbr.c: replaced st_mtim with st_mtime, that's what
  it should have been.  Added #include of h/tws.h to pick up
  dtime() prototype.

Replaced second line of Content-Tyhpe of -attachformat 0 example.

* h/mh.h, h/prototypes.h, uip/mhbuildsbr.c, uip/send.c,
  uip/sendsbr.c, uip/viamail.c, uip/whatnowsbr.c, man/send.man:
  added -attachformat switch to send, to support alternate MIME
  header contents when using -attach.  See send man page for
  description.

* man/mhbuild.man: wrapped one appearance of "Content-Disposition"
  with quotes, to be consistent with others.

* h/utils.h, sbr/utils.c, uip/flist.c, uip/folder.c: Move duplicate
  function num_digits into utils.c

Add missing create_folder prototype.

* sbr/m_draft.c, sbr/utils.c, uip/folder.c, uip/inc.c, uip/mhstoresbr.c,
uip/popi.c, uip/refile.c: Add create_folder() function, replacing duplicate
code during folder creation.

Added support for optional Content_Disposition header in mhbuild directive.s

Added RFC2183 to reference of RFC1806 for Content-Disposition header.

Added -nocontentid (and -contentid, for symmetry) switch to mhbuild.  This allows users to disable generation of the Content-ID: header in MIME messages, in order to placate broken version(s) of Outlook.

remove trailing newlines from components to fix bug with spaces at the end of
Subject/References in replies

use AS_HELP_STRING for formatting help messages

add autoconf magic to support old systems that don't support multibyte charsets

fix bug with insertion of newline being wrong if the num function was used at
the end of the format buffer

Fixed make_bcc_file () to use contents of From: in draft, if draft_from masquerade flag is enabled.

more robust multi-byte/column support for field widths
restore right justification feature

* uip/post.c, uip/spost.c: Move the uptolow macro from spost.c to post.c
  where it is needed.

* sbr/fmt_scan.c: Add the missing wchar.h include

*** empty log message ***

remove remnants of code for MMDF

multiply buffer size by MB_CUR_MAX so multi-byte chars fit

* sbr/fmt_scan.c:PUTSF() Fix an off by one formatting issue.

* sbr/fmt_scan.c: Turn the PUTSF macro into a function capable of handling
multi column characters.

* Remove sbr/strerror.c -- strerror(3) is defined in C89.

* patch #3968: Move the add() function from its own file (add.c) and
  into utils.c. There was also a duplicate add() function in mf.c which
  has been removed.

* mh_xmalloc(), mh_xrealloc(), pwd(): Add comments describing the purpose
  of the function.

Remove sbr/pwd.c file, moving the pwd() function into sbr/utils.c.

* patch #3967: Create a mh_xrealloc function to prevent mistakes when
calling realloc.

* patch #3966: Create a mh_xmalloc function to prevent mistakes when
calling malloc.

Avoid non-portable use of $< outside an inference rule.

Rework detection and handling of ndbm: we now check for a working
(library,header) combination, and configure defines variables for each
which the makefile and C file just use, rather than a bunch of HAVE_foo
switches. As a bonus, we no longer unnecessarily link every binary with
the ndbm library even though only slocal uses it. Also you can now tell
configure where to get the header/library so even if we guess wrong you
can overrule us.

Fix stupid accidental dependence on a bash quirk in previous
configure script change.

Rolled release.

Improve the checks for where to find ndbm (dbm_open etc); should now
work on systems with new gdbm and libdb4.

Change some non-standard formatting in a previous log entry so it matches
the conventions used in other entries.

* Fedora Bug #163760: sbr/context_read.c (context_read): Ensure that the
context is only read once.

Fixed annotate call for new delete argument values.

* uip/sendsbr.c (annoaux): Fix the call to annotate()

*** empty log message ***

Fixed a bug where anno -append put the headers in the wrong place
if applied to a message that didn't contain any headers.

Added a special value of "all" to the -number option that causes
anno -delete to delete all matching components instead of just
the first one.

Added new -preserve and -nopreserve options.  Using -preserve
retains the original last accessed and last modified times on
annotated messages.

Fix the AC_PATH_PROG default when vi isn't found during build.

Rolled version.  Should have done it last time!

Make it clearer that lower-case envariables aren't supposed to be set
by the user.

Special-case an MHCONTEXT of "/dev/null" and don't try to lock it -- some
user scripts may use this to suppress modification of context.

[bug #9813] Don't crash after removing the folder if the context file
has no Current-Folder entry.