[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] option mapping in mailer.py

From: Mathias Weinert <mathias.weinert_at_gfa-net.de>
Date: 2006-05-30 15:52:49 CEST

C. Michael Pilato wrote:
> Mathias Weinert wrote:
> > Hi there,
> >
> > I am still hoping that some will commit the following patch to mailer.py
> > so please let my wishes get true...
> >
> > This patch adds a new parameter to the configuration file which allows
> > to use different characters for splitting the to_addr.
> >
> > This is usefull for using additional name information in the mail
> > addresses. Example:
> >
> > to_addr_split_char = ,
> > to_addr = "The great Wallace" <wallace@atanyaddress>, "little Grommit" <grommit@propablyatthesameaddress>
>
> I find this two-option setup silly. Just make the code smart enough to know
> that email records are comma-delimited, and any commas in the quoted regions
> don't count towards that splitting. I whipped up the following helper
> function for performing such a split, but I'm sure there's probably a
> slicker/quicker way to do it:
>

I wonder if such a function is needed at all. For me it would be enough
to use the comma instead of whitespaces. But that may result in problems
for people updating to this version (which will BTW be the same with your
proposal).

What do you think about keeping it very simple and just changing the
split char from whitespace to comma?

Mathias

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 30 15:53:31 2006

This is an archived mail posted to the Subversion Dev mailing list.