[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


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.

- Julian


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

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.