+1 you are absolutly right... A patch would be nice ;)
Bye, Norman 2010/6/24 Tim-Christian Mundt <d...@tim-erwin.de>: > Hi, > > in IMAP the method MailboxMapper.countMailboxesWithName(mailboxName) is > only ever used in StoreMailboxManager.mailboxExists(mailboxName, > session). There the result is checked for duplicates. Duplicates should > be detected before writing and not when reading, for some stores (e.g. > maildir) it is not even possible to have more than one mailbox with the > same name. So this is superfluous and should should be moved to the > MailboxMapper.existsMailbox(mailboxName) which can provide a more > efficient existence test. > > I think I proposed that before but can't find to reason why it's not > there. Thus I thought I'd check back with you before I change it. Any > objections? Otherwise I'd submit a patch. > > Cheers > Tim > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > For additional commands, e-mail: server-dev-h...@james.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org For additional commands, e-mail: server-dev-h...@james.apache.org
