Various minor cleanups in sbr/m_convert.c.

Removed the unused message flag: DELETED.
Now we can store one user-defines seq more.

s/ALLOW_NEW/ALLOW_BEYOND/

Renamed all standard sequences (e.g. cur->c) and made them globally changeable
The full list:
cur -> c
first -> f
last -> l
next -> n
prev -> p
all -> a
new -> b (mnemonic: one beyond the last)
They are globally changeable in config/config.c
Could be that I'll add the old names again, as convenience aliases.

Inverted the second arg of seq_setunseen(), to be more natural.

Another fix for the strchr() transition: Check for NULL arguments.
Is now the point to reintroduce the brkany() helper? Hmm...

Activated default sequence negation (exclamation mark).

Activated a default unseen sequence; updated man pages to it.

Fixed bug in brkstring(), newly introduced by the switch to strchr().
strchr() treats the trailing null byte surprisingly:
``The terminating null byte is considered to be part of the string.''

Globally defined the names for the standard sequences.
I'll rework the whole code to use them and not have hard-coded names anymore.
The `new' sequence will be called `beyond' sequence (`b') from now on.
The `unseen' sequence will be set by default to `u' from now on.
The sequence negation will be set by default to `!' from now on.

sbr/brkstring.c: replaced static brkany() with call to strchr(); plus cleanups

Fixed minor typo in man page.

Removed remaining lines from the --with-pager=... removal, months ago.

s/vfork/fork/
I truly believe that vfork is wrong. Forking should not be too slow for us.

Relayouted all switch statements: case aligns with switch.
Desired style is:
switch (n) {
case foo:
...
}

Removed ``Face'' support in mhl.
It appears to be about `Face:' headers, but neither Jerry Peek, nor
Ken Hornstein knew more.

Removed references to unused default procs and commented config/config.c

Removed the configuration option NOPUBLICSEQ.
You can change it still, by setting mh_seq to NULL in config/config.c.

Renames: s/+outbox/+sent/ and s/cc:/Cc:/
IMO ``sent'' specifies more exactly what it contains. ``outbox'' would
mean to me that these messages are prepared for delivery, but are not
already sent.

Updated INSTALL and changed `nmh' strings to `mmh'

Reworked the man page system and some man page contents (mmh-intro).
Now it's possible to have two equally named pages in two different sections.
This appeared to be interesting for mmh(X), but I don't use it now. Let's
see if we keep the new system or even rework it to become simpler.

Be quiet! Only talk if you have something useful to say.

Adjusted Makefile to the rename: etc/scan.default -> etc/scan.nmh

Updated all man pages and renamed install-mh.man to mmh.man.

Unimportant stuff.

Replaced C program `install-mh' by shell script `mmh'.
It is updated to the separated directories.

Split nmh dir into mmh dir (~/.mmh) and mail storage (~/Mail).
The profile (~/.mh_profile) became (~/.mmh/profile).

The idea was to separate the configuration files and the data (= messages).

The environment variables changed accordingly:
MH -> MMHP (profile)
MHCONTEXT -> MMHC (context)
new: MMH (mmh dir)

mh-install is not adjusted, yet.

Fixed bug, reported by Phil: Profile entry Draft-Folder was case sensitive.
The components in the profile should be matched case*in*sensitive, always.

Cleanup in m_draft. There was some confusingly unnecessary code.

Minor change in output message

Removed the global alias file
Either use personal alias files or the MTA alias file.

Moved etcpath() from config/config.c to sbr/path.c

Code cosmetic

Adjusted the default scan listing: remove the body preview
The original listing is still available as etc/scan.nmh

Code cosmetics

We have `NULL' in order to not need to write `(char *)0' anymore.

Removed void casts for ignored return values of function calls.
IMO, this distracts from the important without being much worth.

We don't want to use copy() anymore; but kept it for concat() only

s/pfolder/curfolder/g

"." is no longer an alias for "cur"
Don't fear more typing, as "cur" will soon be cut down to "c",
likewise the other predefined sequences.

Removed the option to disable `~user' support (MHRC)

Completely reworked the path convertion functions
Moved everything (from sbr/getfolder.c and sbr/m_maildir.c) into
sbr/path.c, but actually replaced the code almost completely.
See h/prototypes.h for the function changes.
sbr/path.c provides explaining comments on the functions.
None of them allocates memory automatically.

Additionally:
- Like for other ``files'', `inc -audit file' places file relative
  to the cwd, not relative to the mh-dir. This is for consistency.
- Replaced add(foo, NULL) with getcpy(foo), which ist clearer.

Cosmetic refactoring

Cosmetic change

Cosmetic change

Added comment

Simplification: Removed unnecessary variable

The existence of the profile entry Path is already enforced by context_read().
Hence, removed the obscure ``free(path("./", TFOLDER));'' calls.
Still I don't see the sense in these lines, but they won't get executed,
as the condition is checked before (and the program aborts then).
Probably that's obsolete stuff that remained through several reworks.

Code reformating.

Improved pluspath() and used it instead of path() where appropriate.

Refactored sbr/getfolder.c

Introduced FDEF and FCUR for speaking arguments to getfolder().
Maybe it would be better to two separate functions, but I cannot
imagine good names for them ... :-(

Rearranged code in the file.

Refactoring in sbr/path.c.
Especially: Replaced compath() with a more general packpath().

Renamed r1bindex() to mhbasename(), to make its function becomes clear.
All calls to r1bindex() had '/' as second argument. For everything else
one can use strrchr(3) instead of r1bindex().

Renamed ssequal() to isprefix(), because that's what it checks.

Removed defines for obsolete BSD bcopy functions; Use getc() directly.

We don't need macros for only putting two strings next to each other.

Removed references to bboards, as we don't support them anymore.

Broke long lines.

Don't call writev() anymore.
It might be a sexy syscall, but it's not portable and more complex than
the mainstream solution. For sexy source program, go elsewhere!

Removed outc(), which only called putchar(), while having the same signature.
I substituted all outc() calls with calls to putchar().

Removed the space between function names and the opening parenthesis.

Reformated comments and long lines

Removed the draft message in favor for a consistent draft folder facility
- reworked sbr/m_draft.c (of course)
- removed the -draftfolder/-draftmessage/-nodraftfolder switches
- removed the -draft switch from anno and refile
- no more need to question on existing draft message - yay!
Everything is more consistent now. Cornercases went away. :-)

(Also fixed the switch numbering, as already done for other files earlier.)

Fixed broken swit numbering (there are more files to fix)

Removed the whom program, which relayed on post(8) functionality.
We can later add one again, which only expands aliases but does not
require the MTA to support -bv.

Removed the split feature of send.
It would split large messages into several partial ones, using MIME.
But if a message contains one large file attached, it won't solve the
problem. We better go without this feature and leave it to the user.

Removed unneeded sbr code.
memmove() and strerror() removed again. ;-) Mmh takes C89, at least, for
available.

Rearranged whitespace (and comments) in all the code!
The primary goal was to only indent with tabs and not mix them with spaces.
Also, no more tabs should appear past the first printable char on each line.
Now editing the sources will be much easier, no matter which editor one uses,
and rearranging won't break aligned code.
(Beware, the changeset is huge, but the `diff -w' changes are moderate.)

Removed mhn, as it was already replaced by mhlist/mhshow/mhstore.

Removed msh, vmh and wmh.
The latter ones were long time obsolete; now they went away.
Msh has hardly to do with everyday mmh usage, and we don't support bboards
anymore.
This one changeset saves us about 7,000 lines of code. *yay*

Removed unused and not compiling program wmh.
It was added 1986, and is not even mentioned in nmh's history.
The website is (was?) http://wmh.sf.net

Removed obsolete MTS docs from README.developers

Clarification fix in man/mhparam.man, related to -debug.

Reworked LINK, now named `altmsglink', similar to BACKUP_PREFIX.

Moved default file/folder modes from acconfig.h to config/config.c

Removed --with-{editor,pager} configure options.
We keep defaulting to vi and more, but search in PATH at run time.
Nowadays, both programs should be available via PATH everywhere.
Otherwise, the the path in the profile.

Documented -debug switch or mhparam(1)

Moved sendmail item to better place in man page mh-profile(5)

Removed configure option --with-hash-prefix and moved backup-prefix to profile

Renamed configure.in to the more modern name configure.ac

Nicer structure in summary printing of the configure script.

Moved the sendmail path config line from mts.conf to the MH profile
This another step in getting rid of the mts stuff.

Removed TLS support. We don't need it anymore as we stopped talking SMTP
Libcrypto and thelike went away also, as they were needed only for TLS.

Removed SASL support. We don't need it anymore because we talk SMTP no more

Removed the MTS and post; now we always use /usr/sbin/sendmail -t
Mmh won't talk SMTP (nor POP).
Also moved OfficialName() from /mts/smtp/hosts.c to sbr/addrsbr.c.

Removed --with-smtpservers configure option and thelike

Removed all POP support
To retrieve mail from POP servers use external tools, e.g. fetchmail.

We'll always use sendmail (spost) now

Install to /usr/local/mmh

This is the birth of mmh. :-)

Whoops, user & port were in the wrong order so msgchk didn't work for POP
connections.

Updated the developer README with a few notes about the git conversion.

Signed-off-by: Ken Hornstein <kenh@pobox.com>

Signed-off-by: Ken Hornstein <kenh@pobox.com>

More documentation about the hooks support.

Clean up code in inc.c

Clean up the confusing use of a variable 'i' in inc.c.
The case where it was being used as 'an error occurred
which means we should not truncate the mailspool' is now
called 'incerr' and merged with the equivalent 'p' variable
used in the POP-specific code. Cases where it was being
used as a random temporary are still called 'i' but now
declared with a much more restricted scope.
This fixes a compilation warning for some compilers.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Suppress tsort diagnostic waffle

tsort produces waffle to standard error if it detects loops in its input.
For our purposes in ordering object files this is harmless, so just send
it all to /dev/null.

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix unused-function warnings in lex output

Fix the unused-function warnings in the output of lex. In particular, we
silence a warning about yyunput being static but unused by exposing it as
a global function. This costs us a few hundred bytes in the binary but
the other available solutions are flex-specific or more complicated.
YY_NO_UNPUT used to silence this but no longer works in recent flex.
On the other hand YY_NO_INPUT does work to silence warnings about input().

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix used-uninitialised warning in fmt_rfc2047.c

Fix a used-uninitialised warning in fmt_rfc2047.c.
I don't think it can actually be used uninitialised, but the
code is too complicated for the compiler to realise this.

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix used-uninitialised warning in new.c

Fix a used-uninitialised warning in new.c by having the check_folders()
routine return first=last=NULL for empty folder lists. This is a more
sensible API anyway. (It would not actually have been possible for
the code to try to use last while it was uninitialised, as it happens.)

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>