Comment 119 for bug 214366

Revision history for this message
In , Josiah-l (josiah-l) wrote :

(In reply to Ulf Zibis from comment #108)
> (In reply to <email address hidden> 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).