Re: [PATCH] Issue 2213 - setting a max diff size

On 16/01/2009, at 16.56, Greg Stein wrote:

> Smaller patches are definitely better. I'll take a look at these, but
> on this first patch, I noted a style problem: note that all of
> mailer.py uses a space after commas (e.g. in function call params).
> Your patches omit that space, rather than following the style.

Fixed. I think i did notice in my later patches and followed the style,
but i have now go through the other patches and fix any style issues.

JonB

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1029041

On 16/01/2009, at 16.56, Greg Stein wrote:

> Smaller patches are definitely better. I'll take a look at these, but
> on this first patch, I noted a style problem: note that all of
> mailer.py uses a space after commas (e.g. in function call params).
> Your patches omit that space, rather than following the style.

Fixed. I think i did notice in my later patches and followed the style,
but i have now go through the other patches and fix any style issues.

JonB

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1029041