Re: [PATCH] mail address handling and option mapping in mailer.py

On Wed, May 03, 2006 at 11:05:52AM +0200, Mathias Weinert wrote:
> * tools/hook-scripts/mailer/mailer.py
> - new option 'to_addr_split_char' added which allows to use another character but the default whitespaces to separate different mail addresses

I don't know much about mailer.py, so this may be a silly question, but it
might help reviewers if you could summarise why this is a useful feature.

Why would someone ever need to use a non-whitespace separator to provide
a list of email addresses?

> - it's now possible to use capital letters in option values which will be passed through a map

This doesn't look like it's related to the change above, so you're
probably better off sending two separate patches (unless the two changes
really are related).

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed May 3 18:19:11 2006