Comment on attachment 8371072
WIP patch 3

>+  nsAutoString newFolderName(EmptyString());
[Strings default to empty...]

>+    while (containsChild) {
>+      rv = ContainsChildNamed(newFolderName, &containsChild);
>+      NS_ENSURE_SUCCESS(rv, rv);
>+      if (!containsChild)
>+        break;
[Weird use of loop condition.]

>+      newFolderName.AppendLiteral("(");
[Could use Append('(');]

>+confirmDuplicateFolderRename=A folder with that name already exists. Would 
>you like to copy the folder under a different name?
You should include the suggested new folder name in the message, otherwise the 
user will expect to choose the new name.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
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

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to