Re: mail (next/previous)

[Sergey Poznyakoff]

>   The current implementation of mail_next() is broken.
> Will crash and burn(SIGSEGV) if you just hit return on the prompt.

Hmm, didn't notice so far :(

> But what's the right fix.  I thought reading somewhere that
> if you hit return on the mail prompt that it is equivalent
> to "next" which prints the the next mail.  But our implementation
> of "next" just move the cursor does not print.  And we have
> a "previous" counterpart which decrements the cursor.

Next should move the dot to the next non-deleted message and print it.
Previous should behave the same way. As far as I know, all existing
implementations follow this principle. I don't know about just hitting
return, though. It seems a good idea to make it run "next". Maybe
it would be better if it just repeated the last executed command?

Regards,
Sergey