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

Mergeinfo function combine_with_lastrange()

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Thu, 15 Oct 2009 00:43:26 +0100

Paul,

Reviewing r39019 for back-port, I struggled to understand the doc string
of combine_with_lastrange(). It seemed to say bits and pieces about what
it did internally, without saying what its purpose or promises or
requirements were. I read through its implementation and its uses and
re-wrote the doc string as in the attached patch. I found that the
"lastrange" parameter was functionally redundant so I removed it, as it
just made the interface to the function more complex.

Please review the patch if you would.

Thanks.
- Julian

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

Received on 2009-10-15 01:43:59 CEST

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