[Bug 214366]

[UlfZibis]

(In reply to Josiah Bruner [:JosiahOne] from comment #111)
> No. The idea here is very similar to the common "priming read" concept
> taught in introductory CS courses.
In advanced CS practice I learned: No code duplicates, because if there comes a change along, one of the duplicates could be overseen.

> Following such a pattern the code here should be:
If there is no other pattern with higher priority ;-)

> newFolderName.Assign(folderName);
> bool containsChild = true;  // no need to preset it to true
> int counter = 2;            // yes, better than starting with 1 as before

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