dfaure added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in kcompositejob.cpp:102
> Yeah, you are right, but we ever emit on error? It's finish whole composite 
> job (deleteLater will be called)

When a subclass calls this slotResult implementation, they expect the behaviour 
of "finish with an error if the subjob had an error".

But NOT emitResult in the normal case (no error), since we might want to 
continue by creating another subjob.

I'm still missing a proper explanation of what's happening and what shouldn't 
happen.

REPOSITORY
  R244 KCoreAddons

REVISION DETAIL
  https://phabricator.kde.org/D9004

To: jtamate, #frameworks, dfaure, anthonyfieroni
Cc: anthonyfieroni

Reply via email to