[Svnmerge] [PATCH] Indented log patch modified v6

Raman Gupta rocketraman at fastmail.fm
Thu Mar 2 06:26:05 PST 2006


Giovanni Bajo wrote:
> Raman Gupta <rocketraman at fastmail.fm> wrote:
> 
>>> I know you wanted it in a separate commit, but I added Alan's suggestion
>>> for the rstrip(message, \n) + \n because it does what you stated above
>>> and ensures that there will be a terminating newline on the message
>>> before prefix_lines is called. 
> 
> I don't see it in the patch though:
> 
>>> +        if message:
>>> +            logs.append(prefix_lines(LOG_LINE_PREFIX, message) + "\n")
>>> +            for match in LOG_SEPARATOR_RE.findall(message):
>>> +                sep = match[1]
>>> +                if len(sep) > len(longest_sep):
>>> +                    longest_sep = sep
> 

Oops, forgot to do the diff first :-)

Cheers,
Raman
-------------- next part --------------
A non-text attachment was scrubbed...
Name: log-indent-v6.patch
Type: text/x-patch
Size: 3041 bytes
Desc: not available
Url : /pipermail/svnmerge/attachments/20060302/7b016d7e/attachment.bin 


More information about the Svnmerge mailing list