On Mon, May 23, 2005 at 03:53:56PM +0200, Peter Stuge wrote:
> > Well no, more like operator-close.cc should set the depot
> > selected mailbox to 0 so that any futhur depot.getSelected()
> > returns 0.
> 
> Hm. I'm not sure.

Now I am. You're of course completely correct and I'm wearing the
dumstrut. :)


On Mon, May 23, 2005 at 04:04:22PM +0200, Jerry Lundstr�m wrote:
> >I think it looks like Depot::defaultmailbox and
> >Depot::selectedmailbox indicate the Depot type (ie. IMAPdir or
> >Maildir++) rather than the actual folder selected in the mailbox.
> 
> No they are Mailbox superclasses with is inherited by IMAPdir or
> Maildir++ . They reside in a array of Mailbox objects and
> defaultmailbox/selectedmailbox are pointers to an element inside
> that array, check depot.get().

Right. I saw the backends vector but didn't understand how it was
used until I had a closer look at the Mailbox class.


> This should also be applied additional to my previous patch:

resetSelected() can't be const but other than that it works like a
charm!


//Peter

Reply via email to