[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    Re: Reason for crashes when filtering mails
From:       Andreas Gungl <a.gungl () gmx ! de>
Date:       2007-02-25 14:21:34
Message-ID: 200702251521.35039 () gungl-dd ! de
[Download RAW message or body]

Am Sonntag, 25. Februar 2007 schrieb Allen Winter:
> On Sunday 25 February 2007 7:39:38 am Andreas Gungl wrote:
> > Hello,
> >
> > starting with KMail from KDE 3.5.6 I often get crashes when filtering
> > many messages in a folder manually (applying all filters).
> > The problem goes down to the following lines in kmfolder.cpp:
>
> [..snip..]
>
> > Well, if someone can comment on this, I'd like to fix it the right way.
>
> I nagged it earlier on this list, and even provided a patch which is
> attached to http://bugs.kde.org/show_bug.cgi?id=140660
>
> So far nobody has commented, and nobody else has tried my patch AFAIK.
> If you are using the KMail from pim+, then you are using my patch.
>
> Please search for the message "About BR 140660" which may
> be in your mailbox for other info.

Allen, I've had a quick look at your changes. I'm running the 3.5 branch.

The index range is not clear to me. Is 0 (zero) a valid index like in arrays? 
I've seen you skipping idx == 0 in some places which looks strange. If 0 is 
not allowed, why is -1 used to indicate a failing find() ?

After applying your changes in kpim+ to my sources, I've got problems to 
filter the last message in a folder. I've simply moved some spam messages 
into a new folder. Then I expected them being filtered back into the spam 
folder. One message remains. And it is unselected and the preview pane is 
empty.

So I think this is an improvement, but not a real solution.

Andreas
-- 
Kindermode Online - Fashion for Kids
http://www.theofeel.de
_______________________________________________
KMail developers mailing list
KMail-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmail-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic