[Bug 214366]

[Josiah-l]

(In reply to Ulf Zibis from comment #108)
> (In reply to neil at parkwaycc.co.uk from comment #107)
> 
> Hi, why not using for statements:
> 
> > >+    newFolderName.Assign(folderName);
> > >+    bool containsChild = true;
> > >+    for (uint32_t i=2; ; i++) {
> > >+      rv = ContainsChildNamed(newFolderName, &containsChild);
> > >+      NS_ENSURE_SUCCESS(rv, rv);
> > >+      if (!containsChild)
> > >+        break;
> > >+      // This could be localizable but Toolkit is fine without it, see
> > >+      // mozilla/toolkit/content/contentAreaUtils.js::uniqueFile()
> > >+      newFolderName.Assign(folderName);
> > >+      newFolderName.AppendLiteral("(");
> > >+      newFolderName.AppendInt(i);
> > >+      newFolderName.AppendLiteral(")");
> > >+    }

For is never used for such a purpose. That's the point of a
while(condition).

-- 
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla.
https://bugs.launchpad.net/bugs/214366

Title:
  Can't delete a folder if Trash already contains a folder of the same
  name

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/214366/+subscriptions