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

List:       kde-pim
Subject:    Re: [Kde-pim] Review Request: avoid the use of an undefined
From:       Kevin Krammer <kevin.krammer () gmx ! at>
Date:       2011-10-17 18:04:06
Message-ID: 201110172004.06864.kevin.krammer () gmx ! at
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Tuesday, 2011-10-11, Guy Maurel wrote:
> Hello Kevin!
> 
> I found something that doesn't meet your prediction, please look under:
> 
> On Sunday, October 09, 2011 04:46:43 PM Kevin Krammer wrote:
> > On Wednesday, 2011-10-05, Guy Maurel wrote:
> > > I see the problem at file
> > > kdepimlibs/akonadi/specialcollectionshelperjobs.cpp at line 419:
> > > q->emitResult();
> 
> ...
> 
> > With the context you have provided (outboxqueue requesting outbox
> > folder), I did a bit of digging.
> > 
> > The code creating and running the DefaultResourceJob expects it either to
> > deliver all special collections it could find but at least the resource's
> > top level collection so it can create all missing ones.
> > 
> > Returning from DefaultResourceJobPrivate::collectionFetchResult() without
> > an error *and* without a valid top level collection violates that
> > expectation. (A debug build should actually assert in
> > SpecialCollectionsRequestJobPrivate::createRequestedFolders(),
> 
> I never got here!
> Where should it comes from?

In SpecialCollectionsRequestJobPrivate::resourceScanResult when the job has no 
error, either of the two following paths calls createRequestedFolder.

In our case, where we have a DefaultResourceJob, resourceScanResult is 
connected to the job when it is created in 
SpecialCollectionsRequestJobPrivate::lockResult.

Cheers,
Kevin
-- 
Kevin Krammer, KDE developer, xdg-utils developer
KDE user support, developer mentoring

["signature.asc" (application/pgp-signature)]

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/

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