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.

Reply via email to