On 10/10/12 18:42, Thomas Lübking wrote:
I don't get the assert, but run into
    qDebug()<<  "drag-and-drop: mailbox not found";

Are you sure you're building in the debug mode?

Random guess is that the stream contains a false sectionmark and my very first 
random guess for the reason would be

    stream.setVersion(QDataStream::Qt_4_6);

in ../../../../trojita/src/Imap/Model/MsgListModel.cpp, line 335

You're on a right track. I suspect it doesn't hurt to always specify the same version on both ends (and I have to fix the stream version at least in the cache, to make sure that Trojita's on-disk cache doesn't depend on Qt version being used).

There are several problems here:

- Using asserts on code which checks something which can be used-provided (like the drag-and-drop data) is evil -- fixed, my bad. - The model doesn't reimplement mimeData() and the data returned by QAIM's implementation is garbage (how come the MIME type matches, though???). - The model only supports two MIME types, one for passing a list of messages and another one for passing a list of message parts. Another MIME type accepting "anything supported", along with a new method of serialization, was needed. This is now in master (2a59a16). Could you please rebase your change on top of that? There *will* be conflicts.

PS: need to be more careful about the ctrl+enter sending, enter + ctrl+v seems 
to cause it as well...

Could you please file a bug for this in the Trojita's issue tracker [1]? And could you please fix it as well? :)

With kind regards,
Jan

[1] https://projects.flaska.net/projects/trojita/issues/new

--
Trojita, a fast e-mail client -- http://trojita.flaska.net/

Reply via email to