check for linecap and trailing space in mhbuild for text/plain

without this a mail with too long lines can be created, or
trailing space is removed by mhsign.

ignore LENERR2 in seq_read()

this is unnecessary and brings problems because
sequences aren't created with a linecap.

adjust handling of too long lines in m_getfld2

Now a field is complete read, if a line is to long.
Also no length check is done in the body, because
the caller can check f.valuelength.

Fix for commit bd02210b2be64956a952f925a2dcd35fb42f6136

Merge branch 'm_getfld2-meillo' into master

mhsign: Fix key lookup of aliases when encrypting

Now mhsign can handle addresses that contain whitespace (e.g.
``markus schnalke <meillo@marmaro.de>'') in aliases. The helper
tool ap(1) is used to extract the actual addresses.

refile: Never change the current folder

Refile is different to all other tools, as its +folder arguments
are destination folders, not source folders (it even supports
multiple of them). A source folder can be specified with -src.
Refile used to change the current folder to the source folder.
It does not do so anymore.

Most often the current folder is used as the source folder; this is
the default. Hence changing the current folder is a no-op. But if
one refiles from some other source folder, changing to that folder
is seldom useful. Consider the scenario where one wants to collect
the last messages of a set of folders in some other folder. The
old behavior required to know where you are to collect the messages,
because after the first refile you'd be somewhere else:

for i in one two three ; do
refile -src +$i l +lastmsgs
done
# now we are in +three, hence go back where we were
folder +lastmsgs

The new behavior allows to move the messages just here (@), no
matter where that is:

for i in one two three ; do
refile -src +one l @
done
# now we still are where we were

Refiling actions do no more affect the current folder, hence one
may securely refile with different -src +folder many times in one
row, without having to care for changing locations.

refile: Fix an error message; always use braces; add/remove comments

Fix order of (trim) and (decode) in repl*comps

Before MH had (decode), the format escapes used to be:
%(void{comp})%(trim)%(putstr)
i.e. set the internal str register to the component text, trim
trailing whitespace from it, then print the value. Later (putstr)
was simply replaced by (decode), which first RFC-2047 decodes the
string and then prints it (if it is the outermost function). That
almost always produced the correct result. But this way the
whitespace trimming is done at the wrong point. Correct is to first
decode the string and then trim it. tests/scan/test-mh-format
checks for that.

Trim trailing whitespace in scan and show

This commit fixes the most common scan listings. Still trailing
whitespace will be generated by legacy scan listings.

The formatfield workaround in etc/mhl.headers is a bit clumsy.
Maybe, an mhl variable should be defined for this case ... or
better not, to keep the mh-format and the mhl functions orthogonal.

Make default scan/inc listing file-based

There's now a new format file etc/scan.default, which contains
the default scan listing. Previously it was hard-coded as a define
in h/scansbr.h. The default scan listing format can be set in
config/config.h and can be queried with `mhparam scanformat'.

Cleanup manpage mh-format(5)

- More suitable scan listing examples
- Adjust paths
- There are no ``local mh experts'' anymore in the mmh world
- Add overview on the escape types
- %(decode) does print str as well
- Decode Subject header in the example
- Further minor stuff

Enable addr decoding in replcomps and replgroupcomps

Tests: mmh currently generates a part line, so expect it

Fix spelling and encoding errors in manpages and an error message

test also libtinfo for modf

Some Gentoo instalations has this instand of ncurses

Replace getcpy() and strdup() with mh_xstrdup()

this harmonise the code.

Also fix some bugs, if return value of strdup is not checked.

simple mbox support for rcvstore

Some MTA(opensmtpd) add a ``From '' line ad the head of
mails befor handling the mail to a .forwoad MDA. m_getfld2
don't support such mails, so rcvstore should handle this
mbox-like mails.

Replace free() with mh_free0()

mh_free0() free's the memmory and set's the pointer to NULL after
the free. This avoids doublefree and dereferenc of a non valid poiter.

remove unnecessary casts

These casts after malloc just make the code harder to read

Replace mh_xmalloc() with mh_xcalloc()

calloc() is safer, because it initialize the memory to zero.

Call context_read() before getting filter in repl

The manual page of repl says "[i]f a file named `mhl.reply' exists in the
user's mmh directory, it will be used instead of [the default form file]".
Now this behavior is implement.

check return value of m_getfld2 befor using the field

this bug causes a segfault, because the field.value is a
nullptr if FILEEOF2 is returned.

pipe mail throu show on repl

now repl works as expected for MIME-messages

Add existing "From:" to Bcc-ed messages.

If you use multible mailboxes, the correct one is
now used for Bcc-messages.

fix profile path in mhl(1)

Change some "nmh" into "mmh" in man pages

This is to avoid confusion between nmh and mmh

Use boolean instand of bool

Revert "add free_field as standard for struct field"

This is not necesary, because the compieler have to take care
about NULL != (void*)0

This reverts commit a87df3543d3bc128ba4079d1f95638476ba5ca50.

fix lengthcheck in m_getfld2()

The lenght check akcepted to big field names

fix segfault in prompter

This was necesary because m_getfld2() clears now f.value
if FILEEOF2 is reached

add free_field as standard for struct field

this is a easy way to initializ a struct field. ``{{0}}''
can lead to bugs, because NULL is not necesary ``((void *)0)''

m_getfld2 is more tranceparent on error

now all read content of the file is given to the caller,
so he can possible fix this error.

make m_getfld2() aware of CRLF lineendings

make m_getfld2() called with an error state a noop

This probduce some code dublication, but the will be
reduced, if we make m_getfld2() more transparent on errors.

Finally we get rid of the unmaintainable m_getfld()!

m_getfld() was about 700 lines, 300 of them being comments -- that's 42%!
m_getfld2() is 240 lines, 30 of them being comments. That's only half as
much code. Most important, however, is that this code is understandable
and thus maintainable. We escaped the dead end of the heavily optimized
m_getfld() be re-writing it from scratch. (Thanks to Philipp Tacaks!)

m_getfld2() is line-based and not character-based like m_getfld(). This
change spared us to keep digging into stdio internals or implementing
our own I/O buffering. We are now portable to any standard-conforming
stdio implementation.

Adjust tests to m_getfld2() and write some more

m_getfld2() is more strict than m_getfld() and does less guessing.

Only lines consisting of nothing but dashes are body separators

This is how the documentation specifies it, see mh-mail(5). The old
m_getfld(), however, was more relaxed and treated any header line that
starts with a dash as the body separator. We are more strict here,
because ``-foo: bar'' is a valid header line (according to RFC 2822),
that should thus not be treated as the body separator, whereas a line
``------'' in the header could only be a format error if we don't
take it as the body separator.

Convert inc and scan from m_getfld() to m_getfld2()

The main conversion was done in uip/scansbr.c.

Mbox handling is now completely outside of m_getfld2(), which does not
even know about From_ lines.

No more need to call thisisanmbox() (formerly m_unknown(), ``the magic
invocation'') before calling scan().

The newly introduced gotos should be eliminated ...

Convert rcvdist and repl from m_getfld() to m_getfld2()

m_getfld2: Return line for LENERR2, plus some documentation

m_getfld2: ERR2 means a read error, thus name it IOERR2

m_getfld2: Whitespace in component name is forbidden

Also on style: No function calls should be used to initialize
variables in definition statements. Use separate assignments in
this case.

Use continue instead of one long if for the whole loop body

Plus remove unnecessary comments. This improves the signal-noise ratio.
The identifiers are already telling us enough.

Return invalid line for FMTERR in m_getfld2()

This is necessary for mbox handling outside of m_getfld2().

No error-checking for error-checking alloc functions

The mh_x(m|c|re)alloc function always return successful
... or terminate the program on their own, otherwise.

check for (un)selection befor (un)set

If you select or unselect the same msgnum
numsel won't be increased/decreased twice.

handle hghsel lowsel and numsel in (un)set_selected

Now you can use this funkitions without taking care
of this values.

Thanks c_14 for reporting this bug.

No error-checking for error-checking alloc functions

The mh_x(m|c|re)alloc function always return successful
... or terminate the program on their own, otherwise.

Fix minor typo in man page

Adjust the tests to less tolerant m_getfld2()

We require the header and body to be separated by an empty line.
We require header lines to end with a newline. m_getfld2() does
not try to guess as much as the old m_getfld() did.

Convert most of the tools from m_getfld() to m_getfld2()

Add new implementation m_getfld2()

This is mainly Philipp Takacs work.

Fix minor typo in man page

check for (un)selection befor (un)set

If you select or unselect the same msgnum
numsel won't be increased/decreased twice.

handle hghsel lowsel and numsel in (un)set_selected

Now you can use this funkitions without taking care
of this values.

Thanks c_14 for reporting this bug.

Better description of the Sender header in man page

Plus wording improvements and other minor stuff.

Re-add --with-lockdir=DIR configure option

At the same time eliminate acconfig.h (and thus the autoreconf
warning).

Include the lock dir string only if we use dot locking.

More sensible check for 7bit ASCII

Always true condition reported by -Wextra -pendatic.

Remove unused parameter

Reported by -Wextra -pedantic.

Remove dead code: FLDEOF and BODYEOF are never set

Bump version number

This is mmh-0.2

Decode address headers in replcomps

Update README and release howto to new NEWS file

Also reorder the release steps to first check the tarball and
then tag it as final.

Add NEWS file, which summarizes changes per release

Include a static copy of the CVS log in the dist tarball

Teach spost to handle address groups

Plus more tests for address groups.

test Group Addresses in spost

catch unparsable addresses

Now spost don't coredump if an address can't be parsed

test if spost catch bad addresses

test if aliasing work in spost

Fix configure.ac warning

Remove check for not used networking libraries in configure.ac

Mmh does no more contain networking code, hence we don't need this.
This way, we can also get rid of aclocal.m4.

Build the release tarball from git, not manually

Previously the Makefiles contained lists of files to distribute.
They built the release tarball manually. It's much easier to just
use git, which knows which files are the relevant files, because
it's the ones that are under version control. We just need to
add some additional, pre-generated files for convenience, so that
the user does not need to have auto* and lex installed.

The release tarball is now fully assembled in the top-level
Makefile. (The only tricky part is sbr/dtimep.c, because that
needs to be generated in the subdirectory.)

Merge

Include `test' dir in release tarball

The subdirectories `docs' and `test' are managed by the top-level
Makefile now, because there's nothing to do in them but to copy
them for the distribution tarball. (This way, we also don't have
to include each new file in these directories to the file list in
Makefile.in.)

Update lists of files to distribute

Also partly reordered the lists.

initialize mp in ismymbox

Drop the SYNOPSIS section from some man pages

For some man pages this section makes few sense.

Plus some minor formating stuff on the way ...

Minor clarification in mh-alias(5)

Silence the output of `cd -' in the tests

Use cksum(1) instead of md5sum(1) in the tests

cksum(1) is part of POSIX; md5sum(1) is not.

Run the tests in defined (i.e. sorted) order

Merge

consider Default-From as own address

Remove outcommented code (C99 comment)

Our code should be clean C89 (ANSI C). There's no reason to
keep this C99 comment.

Ensure $SHELL is set for the tests

Non-interactive shells might not set it (this is true for dash
and mksh, at least). Currently we use $SHELL in whatnow(1).
Although we likely remove this use, it's rather worthwhile to
have $SHELL available in the test scripts.

Update mmh-intro(7) man page

Remove mh-tailor(5) man page

Split the releasing instructions into own document

This makes them better visible.

Updates README, INSTALL, and docs/MACHINES

Update TODO list

Some comments on the removed items:
- Mmh's show does already invoke a pager
- The reverse action of `rmm' is: refile -src +trash l @
- Disjunct contexts are already possible (just set MMHC accordingly)
- We already generate the mh-chart man page
- Prompter can already be used to input the recipients. Use:
        Editor: prompter
        prompter: -nobody
        prompter-next: vi
- Comp: -nowhatnowproc is obsolete; use `-whatnowproc true' instead.
  (It puts an empty draft into the drafts folder.)
- Forw: Since we use MIME to forward messages, these todos are obsolete
- Mhmail is not considered to be a full-featured interface to MH
- We now do have RFC 2047 encoding of headers

add the mmh mailing list to the doc

remove docs/README.SASL

we don't need it, because smtp support is gone.

add mailsync description

remove TODOs for not supportet programms

send: MIMEify with non-ascii header and ascii body!

This code is not perfect, as it scans body lines twice. We might
want to rework it someday. But, ignoring the minor performance
issue, it ensures, that we MIMEify in the case when some headers
contain non-ASCII text but the body does not.

remove variable mp2 from spost in putadr

This variable is set but not used. Thanks gcc for the warning.

Add DCC support into whom

Plus a test for it.