author | alwin <alwin> | 2003-12-14 01:53:46 (UTC) |
---|---|---|
committer | alwin <alwin> | 2003-12-14 01:53:46 (UTC) |
commit | 898d1d9abda9889c3bb3adc44dd253e0a4ede746 (patch) (unidiff) | |
tree | 8fb7afa554b94c0fe51723f8c2146859a705c2d7 /noncore/net/mail/libmailwrapper | |
parent | 52966e8b529e78ae431839abd90cec2ebb0d6dff (diff) | |
download | opie-898d1d9abda9889c3bb3adc44dd253e0a4ede746.zip opie-898d1d9abda9889c3bb3adc44dd253e0a4ede746.tar.gz opie-898d1d9abda9889c3bb3adc44dd253e0a4ede746.tar.bz2 |
memleak fixed, some obsolet debugs removed
Diffstat (limited to 'noncore/net/mail/libmailwrapper') (more/less context) (ignore whitespace changes)
-rw-r--r-- | noncore/net/mail/libmailwrapper/imapwrapper.cpp | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/noncore/net/mail/libmailwrapper/imapwrapper.cpp b/noncore/net/mail/libmailwrapper/imapwrapper.cpp index a01a2a9..9ee0dff 100644 --- a/noncore/net/mail/libmailwrapper/imapwrapper.cpp +++ b/noncore/net/mail/libmailwrapper/imapwrapper.cpp | |||
@@ -288,27 +288,29 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att) | |||
288 | addresslist = address_list_to_stringlist(head->env_bcc->bcc_list); | 288 | addresslist = address_list_to_stringlist(head->env_bcc->bcc_list); |
289 | m->setBcc(addresslist); | 289 | m->setBcc(addresslist); |
290 | } | 290 | } |
291 | if (head->env_reply_to!=NULL) { | 291 | if (head->env_reply_to!=NULL) { |
292 | addresslist = address_list_to_stringlist(head->env_reply_to->rt_list); | 292 | addresslist = address_list_to_stringlist(head->env_reply_to->rt_list); |
293 | if (addresslist.count()) { | 293 | if (addresslist.count()) { |
294 | m->setReplyto(addresslist.first()); | 294 | m->setReplyto(addresslist.first()); |
295 | } | 295 | } |
296 | } | 296 | } |
297 | m->setMsgid(QString(head->env_message_id)); | 297 | m->setMsgid(QString(head->env_message_id)); |
298 | } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_INTERNALDATE) { | 298 | } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_INTERNALDATE) { |
299 | mailimap_date_time*d = item->att_data.att_static->att_data.att_internal_date; | 299 | mailimap_date_time*d = item->att_data.att_static->att_data.att_internal_date; |
300 | #if 0 | ||
300 | QDateTime da(QDate(d->dt_year,d->dt_month,d->dt_day),QTime(d->dt_hour,d->dt_min,d->dt_sec)); | 301 | QDateTime da(QDate(d->dt_year,d->dt_month,d->dt_day),QTime(d->dt_hour,d->dt_min,d->dt_sec)); |
301 | qDebug("%i %i %i - %i %i %i",d->dt_year,d->dt_month,d->dt_day,d->dt_hour,d->dt_min,d->dt_sec); | 302 | qDebug("%i %i %i - %i %i %i",d->dt_year,d->dt_month,d->dt_day,d->dt_hour,d->dt_min,d->dt_sec); |
302 | qDebug(da.toString()); | 303 | qDebug(da.toString()); |
304 | #endif | ||
303 | } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_RFC822_SIZE) { | 305 | } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_RFC822_SIZE) { |
304 | size = item->att_data.att_static->att_data.att_rfc822_size; | 306 | size = item->att_data.att_static->att_data.att_rfc822_size; |
305 | } | 307 | } |
306 | } | 308 | } |
307 | /* msg is already deleted */ | 309 | /* msg is already deleted */ |
308 | if (mFlags.testBit(FLAG_DELETED) && m) { | 310 | if (mFlags.testBit(FLAG_DELETED) && m) { |
309 | delete m; | 311 | delete m; |
310 | m = 0; | 312 | m = 0; |
311 | } | 313 | } |
312 | if (m) { | 314 | if (m) { |
313 | m->setFlags(mFlags); | 315 | m->setFlags(mFlags); |
314 | m->setMsgsize(size); | 316 | m->setMsgsize(size); |
@@ -709,22 +711,27 @@ void IMAPwrapper::deleteMail(const RecMail&mail) | |||
709 | } | 711 | } |
710 | const char *mb = mail.getMbox().latin1(); | 712 | const char *mb = mail.getMbox().latin1(); |
711 | err = mailimap_select( m_imap, (char*)mb); | 713 | err = mailimap_select( m_imap, (char*)mb); |
712 | if ( err != MAILIMAP_NO_ERROR ) { | 714 | if ( err != MAILIMAP_NO_ERROR ) { |
713 | qDebug("error selecting mailbox for delete: %s",m_imap->imap_response); | 715 | qDebug("error selecting mailbox for delete: %s",m_imap->imap_response); |
714 | return; | 716 | return; |
715 | } | 717 | } |
716 | flist = mailimap_flag_list_new_empty(); | 718 | flist = mailimap_flag_list_new_empty(); |
717 | mailimap_flag_list_add(flist,mailimap_flag_new_deleted()); | 719 | mailimap_flag_list_add(flist,mailimap_flag_new_deleted()); |
718 | store_flags = mailimap_store_att_flags_new_set_flags(flist); | 720 | store_flags = mailimap_store_att_flags_new_set_flags(flist); |
719 | set = mailimap_set_new_single(mail.getNumber()); | 721 | set = mailimap_set_new_single(mail.getNumber()); |
720 | err = mailimap_store(m_imap,set,store_flags); | 722 | err = mailimap_store(m_imap,set,store_flags); |
723 | mailimap_set_free( set ); | ||
724 | mailimap_store_att_flags_free(store_flags); | ||
725 | |||
721 | if (err != MAILIMAP_NO_ERROR) { | 726 | if (err != MAILIMAP_NO_ERROR) { |
722 | qDebug("error deleting mail: %s",m_imap->imap_response); | 727 | qDebug("error deleting mail: %s",m_imap->imap_response); |
723 | return; | 728 | return; |
724 | } | 729 | } |
730 | qDebug("deleting mail: %s",m_imap->imap_response); | ||
731 | /* should we realy do that at this moment? */ | ||
725 | err = mailimap_expunge(m_imap); | 732 | err = mailimap_expunge(m_imap); |
726 | if (err != MAILIMAP_NO_ERROR) { | 733 | if (err != MAILIMAP_NO_ERROR) { |
727 | qDebug("error deleting mail: %s",m_imap->imap_response); | 734 | qDebug("error deleting mail: %s",m_imap->imap_response); |
728 | } | 735 | } |
729 | qDebug("Delete successfull"); | 736 | qDebug("Delete successfull %s",m_imap->imap_response); |
730 | } | 737 | } |