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

Re: svn commit: r1303883 - /subversion/trunk/subversion/tests/cmdline/svntest/main.py

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Thu, 22 Mar 2012 13:10:12 -0400

On 03/22/2012 12:45 PM, gstein_at_apache.org wrote:
> Author: gstein
> Date: Thu Mar 22 16:45:30 2012
> New Revision: 1303883
>
> URL: http://svn.apache.org/viewvc?rev=1303883&view=rev
> Log:
> Followup to 1303727: use .rstrip() rather than whacking a single '\n'
> character off the end. This approach can also handle '\r\n'. Trailing
> whitespace happens to get killed, but no bigg.

You beat me to this change. But for the sake of precision, why not go ahead
and use ".rstrip('\r\n')" ? Is the extra precision just disinteresting in
this context?

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2012-03-22 18:10:49 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.