Hi chaw,
Thanks for the patch.
> please excuse and correct any inconsistencies, omissions, or other
> errors.
Given David asked for another patch submission, I'd prefer not to have
the added ‘else’s given done() doesn't return.
> + if (mp->nummsg == 0) {
> + if (emptysw) {
> + done (0);
> + }
> + else {
> die("no messages in %s", folder);
> + }
> + }
...
> + if ((nmsgs = read_hdrs (mp, datesw)) <= 0) {
> + if (emptysw) {
> + done (0);
> + }
> + else {
> die("no messages to sort");
> + }
> + }
--
Cheers, Ralph.