Fix missing va_end call in uip/mhmisc.c

Fix misspelled variable in uip/whatnow2.sh

This make the error message better

Fix out-of-bounds error when incorporating email from stdin

Before this patch, if +inbox is empty, following error happened:

$ inc -file - < /dev/null

Incorporating new mail into inbox...

inc: no messages incorporated, continuing...
inc: Bug: message out of bounds

This happened due improper call to `seq_setunseen', which implicitly
assumed that there is at least one message (it does not handle case
where both mp->hghmsg and mh->lowmsg are 0).

Bug does not happens with command `inc -file empty', since file
is checked for being empty before any read attempts.

Signed-off-by: Dmitry Bogatov <KAction@gnu.org>

Fix FILE resource leaks in uip/distsbr.c

add Sendmail option to spost manpage

Thanks Dmitry Bogatov

add uip/whatnow2 to .gitignore

Bump version number

This is mmh-0.3

fix whatnow2 send

in the last commit whatnow2 send was changed
to keep the metafile. This don't work with the
``exec send''. Now the send() is renamed to
sendfunktion and the ``exec'' is removed.

del metafile in whatnow2 only if send don't fail

If send fails, you want to keep the metafile. Without this dist wold
only work, if send works on the first time. Also some metainfo don't
get lost.

touch the DISTINCLUDE in mmhdist target

This avoids rebuilding this files. So autoconf and flex
is not need for a normal user.

Add "delete" to "delete" pattern in whatnow2's case construct

The attached patch modifies whatnow2 so that calling `whatnow2 delete`
works as may be expected.  Without this change, the "delete" function
in whatnow2 was called only when whatnow2 was executed as one of the
following.

* whatnow2 del
* whatnow2 dele
* whatnow2 delet

better error handling in whatnow2

if the command is unknown then usage is printed and if usage is
called with an argument other then 0 exit 1 is forced.

fix bug in decode_rfc2047()

If iconv() returns an error and the source encoding
is utf-8, the character jump has an of by one: the
source string is incresed by one possition more the
the length is decreased.

Update comment in script concerning the VCS

We're not using CVS anymore, but a VCS -- doesn't matter which one.

fix dist

Because of always mimeify the dist hasn't work anymore.
This is fixed by skipping attach() and signandenc(), if
it's handled as a redistribution message.

Add more tests to tests/test-send-mimeify

Fix typo in whatnow2 function "display"

The attached patch fixes a typo in the whatnow2's "display" function.
[ -z instand of [ -n for a check if a variable is empty.

add a test for repl on a ASCII only system

This should create the same output as with utf-8.
But the Umlaut should be displayed as '?'. This isn't
the case at the moment.

Fix typo in whatnow2.sh

command call is done with `command` not "command"

Test if MIME creation works

Test if send uses mhbuild only, if it is usefull. A message
already in MIME-Formate should not cause a fail.

unset MM_CHARSET in tests/scan/test-scan-multibyte

If MM_CHARSET is set to anything other then ascii this
test probaly fail, but the test is mean to run on a ascii
only envirement.

Adjust tests/bad-input/test-header for m_gefld2()

With the m_gefld2 change mmh ignores more errors. This causes
a diffrent behaviour in some cases.

Fix tests/mhparam/test-mhparam

The behaviour of readconfig() has changed. This causes
another behaviour for mhparam.

send: Always MIMEify and pass already MIMIfied drafts through

Even if technically not necessary, MIMEify the draft. This simplifies
the code at hardly any cost.

To still be able to create arbitrary complex MIME structures, make
send(1) pass already MIMEified drafts through, without automatically
invoking mhbuild(1) on them a second time (which would fail). This
way, you can manually invoke mhbuild(1) and afterward call send(1)
... and everything will work just as expected.

Also documented this in the man page, and additionally added a
paragraph about the signing and encrypting feature of send(1), which
wasn't documented yet.

mhsign should work now

Allow send(1) to send from folder other than +drafts

This allows having multiple draft folders or queueing drafts
for automatic sending.

some small fixes for whatnow2.sh

some bugs fixes most found by markus

Enable inc(1) to read from stdin

The -file switch handles the argument "-" special and reads
from stdin in this case.

add my whatnow replacement whatnow2

whatnow2 is tool which acts like whatnow, but
don't trap the user in it.

mhsign/mhpgp: Clarify the manpages and usage lines

These tools are shell scripts and thus have a different option
processing mechanism than the other mmh tools. Unfortunately, it
is limited: Options must come first.

Also note that mhpgp accepts a +folder argument, and what the
defaults are.

Update manpages send(1) and mh-draft(7)

Documented the actual behavior and homogenized the style.

Remove extra space from 'From' line when writing to mbox

Fix header lookup table setup in slocal(1)

simplify whatnow.c/main() function

GNU Complexity: 60 -> 59.

use old error message

don't warn if lenerr is ignored

better error handling

only warn for LENERR2, or ignore it in some cases

fork in repl as a tree

This fix the bug in the last improvement (pipe mail through show).
In the first patch repl only waits for one child, now the tree causes
that both childs finished befor repl continues.

fix 2fb1dd2271e5be9d0b0c9cbd4e7d6d5d51aaecb9

Do a length check in the header, but not in the body.

fixup for 2a903a43ad7d26f5581d4ad1bbc077f8ecc5f5a9

a spell error hapend in uip/spost.c: state not status

check for to long lines and ioerror in spost

Now spost stop and exit if some error happend. Befor this
spost stoped on an error in the body and submitted a half
mail.

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.