> -  my $timeout = $self->config('timeout');
> +  my $timeout = $self->config('timeoutsmtpd') # qmail smtpd control file
> +    or $self->config('timeout')               # qpsmtpd override
> +    or 1200;                                  # default smtpd value

Please don't use C< or >.  use C< || > unless you really need
C<or>. (They have different precedence levels.)

In this particular case, it doesn't make a difference, but I've seen
too many cases of people making a royal mess when they try and change
code that uses C<or>.

Anyway, applied with my changes.  :)

-R

Reply via email to