[Bug 214366]

[Josiah-l]

No. The idea here is very similar to the common "priming read" concept
taught in introductory CS courses. Following such a pattern the code
here should be:

newFolderName.Assign(folderName);
bool containsChild = true;
int counter = 2;

//Priming Read
rv = ContainsChildNamed(newFolderName, &containsChild);
NS_ENSURE_SUCCESS(rv, rv);

while (containsChild) {
  // 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(")");

  //Read new data
  rv = ContainsChildNamed(newFolderName, &containsChild);
  NS_ENSURE_SUCCESS(rv, rv);
}

-- 
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