Fixing issue in which for large patches, files fail to revert....

Fixing issue in which for large patches, files fail to revert.
Flaw in loop logic, causes it to miss the occasional file.

BUG=None
TEST=None
TBR=maruel


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26427

[bradn, 2009-09-17 01:22:46]

You just went idle, so I'm gonna submit this one TBR.-BradN

On Wed, Sep 16, 2009 at 6:16 PM, <bradnelson@google.com> wrote:

> Reviewers: Marc-Antoine Ruel,
>
> Description:
> Fixing issue in which for large patches, files fail to revert.
>
> BUG=None
> TEST=None
>
>
> Please review this at http://codereview.chromium.org/209008
>
> SVN Base: svn://chrome-svn/chrome/trunk/tools/depot_tools/
>
> Affected files:
>  M     gclient_scm.py
>
>
> Index: gclient_scm.py
> ===================================================================
> --- gclient_scm.py      (revision 26425)
> +++ gclient_scm.py      (working copy)
> @@ -249,8 +249,8 @@
>         if accumulated_length and accumulated_length + len(p) > 3072:
>           RunSVN(command + accumulated_paths,
>                  os.path.join(self._root_dir, self.relpath))
> -          accumulated_paths = []
> -          accumulated_length = 0
> +          accumulated_paths = [p]
> +          accumulated_length = len(p)
>         else:
>           accumulated_paths.append(p)
>           accumulated_length += len(p)
>
>
>

[M-A Ruel, 2009-09-17 10:58:51]

oops, thanks!

[M-A Ruel, 2009-09-17 11:00:03]

FYI