6 files changed, 158 insertions, 98 deletions
diff --git a/noncore/net/mail/accountview.cpp b/noncore/net/mail/accountview.cpp
index 6963027..c7b1eeb 100644
--- a/noncore/net/mail/accountview.cpp
+++ b/noncore/net/mail/accountview.cpp
@@ -1,132 +1,132 @@
 #include "accountview.h"
 #include "imapwrapper.h"
 #include "pop3wrapper.h"
 #include "mailtypes.h"
 #include "defines.h"
 /**
 * POP3 Account stuff
 */
 POP3viewItem::POP3viewItem( POP3account *a, QListView *parent )
    : AccountViewItem( parent )
 {
    account = a;
    wrapper = new POP3wrapper( account );
    setPixmap( 0, PIXMAP_POP3FOLDER );
    setText( 0, account->getAccountName() );
 }
 POP3viewItem::~POP3viewItem()
 {
    delete wrapper;
 }
 void POP3viewItem::refresh( QList<RecMail> &target )
 {
    qDebug( "POP3: refresh" );
    wrapper->listMessages( target );
 }
-RecBody POP3viewItem::fetchBody( const RecMail & )
+RecBody POP3viewItem::fetchBody( const RecMail &mail )
 {
-    qDebug( "POP3 fetchBody: IMPLEMENT ME!!" );
+    qDebug( "POP3 fetchBody" );
-    return RecBody();
+    return wrapper->fetchBody( mail );
 }
 /**
 * IMAP Account stuff
 */
 IMAPviewItem::IMAPviewItem( IMAPaccount *a, QListView *parent )
    : AccountViewItem( parent )
 {
    account = a;
    wrapper = new IMAPwrapper( account );
    setPixmap( 0, PIXMAP_IMAPFOLDER );
    setText( 0, account->getAccountName() );
    setOpen( true );
 }
 IMAPviewItem::~IMAPviewItem()
 {
    delete wrapper;
 }
 IMAPwrapper *IMAPviewItem::getWrapper()
 {
    return wrapper;
 }
 void IMAPviewItem::refresh(QList<RecMail>&)
 {
    QList<IMAPFolder> *folders = wrapper->listFolders();
    
    QListViewItem *child = firstChild();
    while ( child ) {
        QListViewItem *tmp = child;
        child = child->nextSibling();
        delete tmp;
    }
    
    IMAPFolder *it;
    for ( it = folders->first(); it; it = folders->next() ) {
        (void) new IMAPfolderItem( it, this );
    }
 }
 RecBody IMAPviewItem::fetchBody(const RecMail&)
 {
    return RecBody();
 }
 IMAPfolderItem::~IMAPfolderItem() 
 {
    delete folder;
 }
 IMAPfolderItem::IMAPfolderItem( IMAPFolder *folderInit, IMAPviewItem *parent )
    : AccountViewItem( parent )
 {
    folder = folderInit;
    imap = parent;
    setPixmap( 0, PIXMAP_IMAPFOLDER );
    setText( 0, folder->getDisplayName() );
 }
 void IMAPfolderItem::refresh(QList<RecMail>&target) 
 {
    imap->getWrapper()->listMessages( folder->getName(),target );
 }
 RecBody IMAPfolderItem::fetchBody(const RecMail&aMail)
 {
    return imap->getWrapper()->fetchBody(aMail);
 }
 /**
 * Generic stuff
 */
 AccountView::AccountView( QWidget *parent, const char *name, WFlags flags )
    : QListView( parent, name, flags )
 {
    connect( this, SIGNAL( clicked( QListViewItem * ) ),
             SLOT( refresh( QListViewItem * ) ) );
 }
 void AccountView::populate( QList<Account> list )
 {
    clear();
    
    Account *it;
    for ( it = list.first(); it; it = list.next() ) {
        if ( it->getType().compare( "IMAP" ) == 0 ) {
            IMAPaccount *imap = static_cast<IMAPaccount *>(it);
            qDebug( "added IMAP " + imap->getAccountName() );
            (void) new IMAPviewItem( imap, this );
        } else if ( it->getType().compare( "POP3" ) == 0 ) {
            POP3account *pop3 = static_cast<POP3account *>(it);
            qDebug( "added POP3 " + pop3->getAccountName() );
diff --git a/noncore/net/mail/accountview.h b/noncore/net/mail/accountview.h
index fe832ec..4cac673 100644
--- a/noncore/net/mail/accountview.h
+++ b/noncore/net/mail/accountview.h
@@ -1,88 +1,88 @@
 #ifndef ACCOUNTVIEW_H
 #define ACCOUNTVIEW_H
 #include <qlistview.h>
 #include <qlist.h>
 #include "settings.h"
 #include "mailwrapper.h"
 class IMAPwrapper;
 class POP3wrapper;
 class RecMail;
 class RecBody;
 class AccountViewItem : public QListViewItem
 {
        
 public:
    AccountViewItem( QListView *parent ) : QListViewItem( parent ) {}
    AccountViewItem( QListViewItem *parent ) : QListViewItem( parent ) {}
    virtual void refresh(QList<RecMail>&)=0;
    virtual RecBody fetchBody(const RecMail&)=0;
 };
 class POP3viewItem : public AccountViewItem
 {
 public:
    POP3viewItem( POP3account *a, QListView *parent );
    ~POP3viewItem();
    virtual void refresh( QList<RecMail> &target );
-    virtual RecBody fetchBody(const RecMail & );
+    virtual RecBody fetchBody( const RecMail &mail );
 private:
    POP3account *account;
    POP3wrapper *wrapper;
 };
 class IMAPviewItem : public AccountViewItem
 {
 public:
    IMAPviewItem( IMAPaccount *a, QListView *parent );
    ~IMAPviewItem();
    virtual void refresh(QList<RecMail>&);
    virtual RecBody fetchBody(const RecMail&);
    IMAPwrapper *getWrapper();
 private:
    IMAPaccount *account;
    IMAPwrapper *wrapper;
    
 };
 class IMAPfolderItem : public AccountViewItem
 {
 public:
    IMAPfolderItem( IMAPFolder *folder, IMAPviewItem *parent );
    ~IMAPfolderItem();
    virtual void refresh(QList<RecMail>&);
    virtual RecBody fetchBody(const RecMail&);
 private:
    IMAPFolder *folder;
    IMAPviewItem *imap;
    
 };
 class AccountView : public QListView
 {
    Q_OBJECT
 public:
    AccountView( QWidget *parent = 0, const char *name = 0, WFlags flags = 0 );
    void populate( QList<Account> list );
    RecBody fetchBody(const RecMail&aMail);
 public slots:
    void refreshAll();
    void refresh(QListViewItem *item);
    
 signals:
    void refreshMailview(QList<RecMail>*);    
 };
 #endif
diff --git a/noncore/net/mail/libmailwrapper/pop3wrapper.cpp b/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
index 633faf6..abb5a42 100644
--- a/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/pop3wrapper.cpp
@@ -1,243 +1,270 @@
 #include "pop3wrapper.h"
 #include "mailtypes.h"
 #include <libetpan/mailpop3.h>
 POP3wrapper::POP3wrapper( POP3account *a )
 {
    account = a;
    m_pop3 = NULL;
 }
 POP3wrapper::~POP3wrapper()
 {
    logout();
 }
 void POP3wrapper::pop3_progress( size_t current, size_t maximum )
 {
    qDebug( "POP3: %i of %i", current, maximum );
 }
+RecBody POP3wrapper::fetchBody( const RecMail &mail )
+{
+    int err = MAILPOP3_NO_ERROR;
+    char *message;
+    size_t length;
+    login();
+    if ( !m_pop3 ) return RecBody();
+    
+    err = mailpop3_retr( m_pop3, mail.getNumber(), &message, &length );
+    if ( err != MAILPOP3_NO_ERROR ) {
+        qDebug( "POP3: error retrieving body with index %i", mail.getNumber() );
+        logout();
+        return RecBody();
+    }
+    
+    logout();
+    return parseBody( message );
+}
+RecBody POP3wrapper::parseBody( const char *message )
+{
+    int err = MAILIMF_NO_ERROR;
+    size_t curTok;
+    mailimf_message *result;
+    RecBody body;
+    
+    err = mailimf_message_parse( (char *) message, strlen( message ), &curTok, &result );
+    if ( err != MAILIMF_NO_ERROR ) return body;
+    
+    if ( result && result->msg_body && result->msg_body->bd_text ) {
+        qDebug( "POP3: bodytext found" );
+        // TODO: why does this line segfault???? gdb says segfault in strlen(), maybe a bug in libetpan.
+        body.setBodytext( QString( result->msg_body->bd_text ) );
+    }
+    return body;
+}
 void POP3wrapper::listMessages( QList<RecMail> &target )
 {
    int err = MAILPOP3_NO_ERROR;
    char *header;
    size_t length;
    carray *messages;
    login();
    if (!m_pop3) return;
    mailpop3_list( m_pop3, &messages );
    for ( int i = carray_count( messages ); i > 0; i-- ) {
        mailpop3_msg_info *info = (mailpop3_msg_info *) carray_get( messages, i - 1 );
        err = mailpop3_header( m_pop3, info->msg_index, &header, &length );
        if ( err != MAILPOP3_NO_ERROR ) {
            qDebug( "POP3: error retrieving header msgid: %i", info->msg_index );
            logout();
            return;
        }
        RecMail *mail = parseHeader( header );
        mail->setNumber( info->msg_index );
        target.append( mail );
    }
    logout();
 }
 RecMail *POP3wrapper::parseHeader( const char *header )
 {
    int err = MAILIMF_NO_ERROR;
    size_t curTok;
    RecMail *mail = new RecMail();
    mailimf_fields *fields;
    
    err = mailimf_fields_parse( (char *) header, strlen( header ), &curTok, &fields );
    for ( clistiter *current = clist_begin( fields->fld_list ); current != NULL; current = current->next ) {
        mailimf_field *field = (mailimf_field *) current->data;
        switch ( field->fld_type ) {
            case MAILIMF_FIELD_FROM:
-                mail->setFrom( *parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
+                mail->setFrom( parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
                break;
            case MAILIMF_FIELD_TO:
-                mail->setTo( *parseAddressList( field->fld_data.fld_to->to_addr_list ) );
+                mail->setTo( parseAddressList( field->fld_data.fld_to->to_addr_list ) );
                break;
            case MAILIMF_FIELD_CC:
-                mail->setCC( *parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
+                mail->setCC( parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
                break;
            case MAILIMF_FIELD_BCC:
-                mail->setBcc( *parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
+                mail->setBcc( parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
                break;
            case MAILIMF_FIELD_SUBJECT:
                mail->setSubject( QString( field->fld_data.fld_subject->sbj_value ) );
                break;
            case MAILIMF_FIELD_ORIG_DATE:
-                mail->setDate( *parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
+                mail->setDate( parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
                break;
            default:
                break;
        }
    }
    
    return mail;
 }
-QString *POP3wrapper::parseDateTime( mailimf_date_time *date )
+QString POP3wrapper::parseDateTime( mailimf_date_time *date )
 {
    char tmp[23];
    
    snprintf( tmp, 23,  "%02i.%02i.%04i %02i:%02i:%02i %+05i", 
        date->dt_day, date->dt_month, date->dt_year, date->dt_hour, date->dt_min, date->dt_sec, date->dt_zone );
    
-    QString *result = new QString( tmp );
+    return QString( tmp );
-    return result;
 }
-QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
+QString POP3wrapper::parseAddressList( mailimf_address_list *list )
 {
-    QString *result = new QString( "" );
+    QString result( "" );
    bool first = true;
    for ( clistiter *current = clist_begin( list->ad_list ); current != NULL; current = current->next ) {
        mailimf_address *addr = (mailimf_address *) current->data;
        
        if ( !first ) {
-            result->append( "," );
+            result.append( "," );
        } else {
            first = false;
        }
-        QString *tmp;
-        
        switch ( addr->ad_type ) {
            case MAILIMF_ADDRESS_MAILBOX:
-                tmp = parseMailbox( addr->ad_data.ad_mailbox );
+                result.append( parseMailbox( addr->ad_data.ad_mailbox ) );
-                result->append( *tmp );
-                delete tmp;
                break;
            case MAILIMF_ADDRESS_GROUP:
-                tmp = parseGroup( addr->ad_data.ad_group );
+                result.append( parseGroup( addr->ad_data.ad_group ) );
-                result->append( *tmp );
-                delete tmp;
                break;
            default:
                qDebug( "POP3: unkown mailimf address type" );
                break;
        }
    }
    
    return result;
 }
-QString *POP3wrapper::parseGroup( mailimf_group *group )
+QString POP3wrapper::parseGroup( mailimf_group *group )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
-    result->append( group->grp_display_name );
+    result.append( group->grp_display_name );
-    result->append( ": " );
+    result.append( ": " );
    if ( group->grp_mb_list != NULL ) {
-        QString *tmp = parseMailboxList( group->grp_mb_list );
+        result.append( parseMailboxList( group->grp_mb_list ) );
-        result->append( *tmp );
-        delete tmp;
    }
-    result->append( ";" );
+    result.append( ";" );
    
    return result;
 }
-QString *POP3wrapper::parseMailbox( mailimf_mailbox *box )
+QString POP3wrapper::parseMailbox( mailimf_mailbox *box )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
    if ( box->mb_display_name == NULL ) {
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
    } else {
-        result->append( box->mb_display_name );
+        result.append( box->mb_display_name );
-        result->append( " <" );
+        result.append( " <" );
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
-        result->append( ">" );
+        result.append( ">" );
    }
        
    return result;
 }
-QString *POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
+QString POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
    bool first = true;
    for ( clistiter *current = clist_begin( list->mb_list ); current != NULL; current = current->next ) {
        mailimf_mailbox *box = (mailimf_mailbox *) current->data;
        if ( !first ) {
-            result->append( "," );
+            result.append( "," );
        } else {
            first = false;
        }
       
-        QString *tmp = parseMailbox( box );
+        result.append( parseMailbox( box ) );
-        result->append( *tmp );
-        delete tmp;
    }
    
    return result;
 }
 void POP3wrapper::login()
 {
    if ( m_pop3 != NULL ) logout();
    const char *server, *user, *pass;
    uint16_t port;
    int err = MAILPOP3_NO_ERROR;
    server = account->getServer().latin1();
    port = account->getPort().toUInt();
    user = account->getUser().latin1();
    pass = account->getPassword().latin1();
    m_pop3 = mailpop3_new( 200, &pop3_progress );
    // connect
    if (account->getSSL()) {
        err = mailpop3_ssl_connect( m_pop3, (char*)server, port );
    } else {
        err = mailpop3_socket_connect( m_pop3, (char*)server, port );
    }
    if ( err != MAILPOP3_NO_ERROR ) {
        qDebug( "pop3: error connecting to %s\n reason: %s", server,
                m_pop3->pop3_response );
        mailpop3_free( m_pop3 );
        m_pop3 = NULL;
        return;
    }
    qDebug( "POP3: connected!" );
    // login
    // TODO: decide if apop or plain login should be used
    err = mailpop3_login( m_pop3, (char *) user, (char *) pass );
    if ( err != MAILPOP3_NO_ERROR ) {
        qDebug( "pop3: error logging in: %s", m_pop3->pop3_response );
        logout();
        return;
    }
    qDebug( "POP3: logged in!" );
 }
 void POP3wrapper::logout()
 {
    int err = MAILPOP3_NO_ERROR;
    if ( m_pop3 == NULL ) return;
    err = mailpop3_quit( m_pop3 );
    mailpop3_free( m_pop3 );
    m_pop3 = NULL;
 }
diff --git a/noncore/net/mail/libmailwrapper/pop3wrapper.h b/noncore/net/mail/libmailwrapper/pop3wrapper.h
index 4e4abad..995bed0 100644
--- a/noncore/net/mail/libmailwrapper/pop3wrapper.h
+++ b/noncore/net/mail/libmailwrapper/pop3wrapper.h
@@ -1,35 +1,38 @@
 #ifndef __POP3WRAPPER
 #define __POP3WRAPPER
 #include "mailwrapper.h"
 class RecMail;
+class RecBody;
 struct mailpop3;
 class POP3wrapper : public QObject
 {
    Q_OBJECT
 public:
    POP3wrapper( POP3account *a );
    virtual ~POP3wrapper();
    void listMessages( QList<RecMail> &target );
+    RecBody fetchBody( const RecMail &mail );
    static void pop3_progress( size_t current, size_t maximum );
 protected:
    void login();
    void logout();
 private:
    RecMail *parseHeader( const char *header );
-    QString *parseMailboxList( mailimf_mailbox_list *list );
+    RecBody parseBody( const char *message );
-    QString *parseMailbox( mailimf_mailbox *box );
+    QString parseMailboxList( mailimf_mailbox_list *list );
-    QString *parseGroup( mailimf_group *group );
+    QString parseMailbox( mailimf_mailbox *box );
-    QString *parseAddressList( mailimf_address_list *list );
+    QString parseGroup( mailimf_group *group );
-    QString *parseDateTime( mailimf_date_time *date );
+    QString parseAddressList( mailimf_address_list *list );
+    QString parseDateTime( mailimf_date_time *date );
    POP3account *account;
    mailpop3 *m_pop3;
 };
 #endif
diff --git a/noncore/net/mail/pop3wrapper.cpp b/noncore/net/mail/pop3wrapper.cpp
index 633faf6..abb5a42 100644
--- a/noncore/net/mail/pop3wrapper.cpp
+++ b/noncore/net/mail/pop3wrapper.cpp
@@ -1,243 +1,270 @@
 #include "pop3wrapper.h"
 #include "mailtypes.h"
 #include <libetpan/mailpop3.h>
 POP3wrapper::POP3wrapper( POP3account *a )
 {
    account = a;
    m_pop3 = NULL;
 }
 POP3wrapper::~POP3wrapper()
 {
    logout();
 }
 void POP3wrapper::pop3_progress( size_t current, size_t maximum )
 {
    qDebug( "POP3: %i of %i", current, maximum );
 }
+RecBody POP3wrapper::fetchBody( const RecMail &mail )
+{
+    int err = MAILPOP3_NO_ERROR;
+    char *message;
+    size_t length;
+    login();
+    if ( !m_pop3 ) return RecBody();
+    
+    err = mailpop3_retr( m_pop3, mail.getNumber(), &message, &length );
+    if ( err != MAILPOP3_NO_ERROR ) {
+        qDebug( "POP3: error retrieving body with index %i", mail.getNumber() );
+        logout();
+        return RecBody();
+    }
+    
+    logout();
+    return parseBody( message );
+}
+RecBody POP3wrapper::parseBody( const char *message )
+{
+    int err = MAILIMF_NO_ERROR;
+    size_t curTok;
+    mailimf_message *result;
+    RecBody body;
+    
+    err = mailimf_message_parse( (char *) message, strlen( message ), &curTok, &result );
+    if ( err != MAILIMF_NO_ERROR ) return body;
+    
+    if ( result && result->msg_body && result->msg_body->bd_text ) {
+        qDebug( "POP3: bodytext found" );
+        // TODO: why does this line segfault???? gdb says segfault in strlen(), maybe a bug in libetpan.
+        body.setBodytext( QString( result->msg_body->bd_text ) );
+    }
+    return body;
+}
 void POP3wrapper::listMessages( QList<RecMail> &target )
 {
    int err = MAILPOP3_NO_ERROR;
    char *header;
    size_t length;
    carray *messages;
    login();
    if (!m_pop3) return;
    mailpop3_list( m_pop3, &messages );
    for ( int i = carray_count( messages ); i > 0; i-- ) {
        mailpop3_msg_info *info = (mailpop3_msg_info *) carray_get( messages, i - 1 );
        err = mailpop3_header( m_pop3, info->msg_index, &header, &length );
        if ( err != MAILPOP3_NO_ERROR ) {
            qDebug( "POP3: error retrieving header msgid: %i", info->msg_index );
            logout();
            return;
        }
        RecMail *mail = parseHeader( header );
        mail->setNumber( info->msg_index );
        target.append( mail );
    }
    logout();
 }
 RecMail *POP3wrapper::parseHeader( const char *header )
 {
    int err = MAILIMF_NO_ERROR;
    size_t curTok;
    RecMail *mail = new RecMail();
    mailimf_fields *fields;
    
    err = mailimf_fields_parse( (char *) header, strlen( header ), &curTok, &fields );
    for ( clistiter *current = clist_begin( fields->fld_list ); current != NULL; current = current->next ) {
        mailimf_field *field = (mailimf_field *) current->data;
        switch ( field->fld_type ) {
            case MAILIMF_FIELD_FROM:
-                mail->setFrom( *parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
+                mail->setFrom( parseMailboxList( field->fld_data.fld_from->frm_mb_list ) );
                break;
            case MAILIMF_FIELD_TO:
-                mail->setTo( *parseAddressList( field->fld_data.fld_to->to_addr_list ) );
+                mail->setTo( parseAddressList( field->fld_data.fld_to->to_addr_list ) );
                break;
            case MAILIMF_FIELD_CC:
-                mail->setCC( *parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
+                mail->setCC( parseAddressList( field->fld_data.fld_cc->cc_addr_list ) );
                break;
            case MAILIMF_FIELD_BCC:
-                mail->setBcc( *parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
+                mail->setBcc( parseAddressList( field->fld_data.fld_bcc->bcc_addr_list ) );
                break;
            case MAILIMF_FIELD_SUBJECT:
                mail->setSubject( QString( field->fld_data.fld_subject->sbj_value ) );
                break;
            case MAILIMF_FIELD_ORIG_DATE:
-                mail->setDate( *parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
+                mail->setDate( parseDateTime( field->fld_data.fld_orig_date->dt_date_time ) );
                break;
            default:
                break;
        }
    }
    
    return mail;
 }
-QString *POP3wrapper::parseDateTime( mailimf_date_time *date )
+QString POP3wrapper::parseDateTime( mailimf_date_time *date )
 {
    char tmp[23];
    
    snprintf( tmp, 23,  "%02i.%02i.%04i %02i:%02i:%02i %+05i", 
        date->dt_day, date->dt_month, date->dt_year, date->dt_hour, date->dt_min, date->dt_sec, date->dt_zone );
    
-    QString *result = new QString( tmp );
+    return QString( tmp );
-    return result;
 }
-QString *POP3wrapper::parseAddressList( mailimf_address_list *list )
+QString POP3wrapper::parseAddressList( mailimf_address_list *list )
 {
-    QString *result = new QString( "" );
+    QString result( "" );
    bool first = true;
    for ( clistiter *current = clist_begin( list->ad_list ); current != NULL; current = current->next ) {
        mailimf_address *addr = (mailimf_address *) current->data;
        
        if ( !first ) {
-            result->append( "," );
+            result.append( "," );
        } else {
            first = false;
        }
-        QString *tmp;
-        
        switch ( addr->ad_type ) {
            case MAILIMF_ADDRESS_MAILBOX:
-                tmp = parseMailbox( addr->ad_data.ad_mailbox );
+                result.append( parseMailbox( addr->ad_data.ad_mailbox ) );
-                result->append( *tmp );
-                delete tmp;
                break;
            case MAILIMF_ADDRESS_GROUP:
-                tmp = parseGroup( addr->ad_data.ad_group );
+                result.append( parseGroup( addr->ad_data.ad_group ) );
-                result->append( *tmp );
-                delete tmp;
                break;
            default:
                qDebug( "POP3: unkown mailimf address type" );
                break;
        }
    }
    
    return result;
 }
-QString *POP3wrapper::parseGroup( mailimf_group *group )
+QString POP3wrapper::parseGroup( mailimf_group *group )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
-    result->append( group->grp_display_name );
+    result.append( group->grp_display_name );
-    result->append( ": " );
+    result.append( ": " );
    if ( group->grp_mb_list != NULL ) {
-        QString *tmp = parseMailboxList( group->grp_mb_list );
+        result.append( parseMailboxList( group->grp_mb_list ) );
-        result->append( *tmp );
-        delete tmp;
    }
-    result->append( ";" );
+    result.append( ";" );
    
    return result;
 }
-QString *POP3wrapper::parseMailbox( mailimf_mailbox *box )
+QString POP3wrapper::parseMailbox( mailimf_mailbox *box )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
    if ( box->mb_display_name == NULL ) {
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
    } else {
-        result->append( box->mb_display_name );
+        result.append( box->mb_display_name );
-        result->append( " <" );
+        result.append( " <" );
-        result->append( box->mb_addr_spec );
+        result.append( box->mb_addr_spec );
-        result->append( ">" );
+        result.append( ">" );
    }
        
    return result;
 }
-QString *POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
+QString POP3wrapper::parseMailboxList( mailimf_mailbox_list *list )
 {
-    QString *result =  new QString( "" );
+    QString result( "" );
    bool first = true;
    for ( clistiter *current = clist_begin( list->mb_list ); current != NULL; current = current->next ) {
        mailimf_mailbox *box = (mailimf_mailbox *) current->data;
        if ( !first ) {
-            result->append( "," );
+            result.append( "," );
        } else {
            first = false;
        }
       
-        QString *tmp = parseMailbox( box );
+        result.append( parseMailbox( box ) );
-        result->append( *tmp );
-        delete tmp;
    }
    
    return result;
 }
 void POP3wrapper::login()
 {
    if ( m_pop3 != NULL ) logout();
    const char *server, *user, *pass;
    uint16_t port;
    int err = MAILPOP3_NO_ERROR;
    server = account->getServer().latin1();
    port = account->getPort().toUInt();
    user = account->getUser().latin1();
    pass = account->getPassword().latin1();
    m_pop3 = mailpop3_new( 200, &pop3_progress );
    // connect
    if (account->getSSL()) {
        err = mailpop3_ssl_connect( m_pop3, (char*)server, port );
    } else {
        err = mailpop3_socket_connect( m_pop3, (char*)server, port );
    }
    if ( err != MAILPOP3_NO_ERROR ) {
        qDebug( "pop3: error connecting to %s\n reason: %s", server,
                m_pop3->pop3_response );
        mailpop3_free( m_pop3 );
        m_pop3 = NULL;
        return;
    }
    qDebug( "POP3: connected!" );
    // login
    // TODO: decide if apop or plain login should be used
    err = mailpop3_login( m_pop3, (char *) user, (char *) pass );
    if ( err != MAILPOP3_NO_ERROR ) {
        qDebug( "pop3: error logging in: %s", m_pop3->pop3_response );
        logout();
        return;
    }
    qDebug( "POP3: logged in!" );
 }
 void POP3wrapper::logout()
 {
    int err = MAILPOP3_NO_ERROR;
    if ( m_pop3 == NULL ) return;
    err = mailpop3_quit( m_pop3 );
    mailpop3_free( m_pop3 );
    m_pop3 = NULL;
 }
diff --git a/noncore/net/mail/pop3wrapper.h b/noncore/net/mail/pop3wrapper.h
index 4e4abad..995bed0 100644
--- a/noncore/net/mail/pop3wrapper.h
+++ b/noncore/net/mail/pop3wrapper.h
@@ -1,35 +1,38 @@
 #ifndef __POP3WRAPPER
 #define __POP3WRAPPER
 #include "mailwrapper.h"
 class RecMail;
+class RecBody;
 struct mailpop3;
 class POP3wrapper : public QObject
 {
    Q_OBJECT
 public:
    POP3wrapper( POP3account *a );
    virtual ~POP3wrapper();
    void listMessages( QList<RecMail> &target );
+    RecBody fetchBody( const RecMail &mail );
    static void pop3_progress( size_t current, size_t maximum );
 protected:
    void login();
    void logout();
 private:
    RecMail *parseHeader( const char *header );
-    QString *parseMailboxList( mailimf_mailbox_list *list );
+    RecBody parseBody( const char *message );
-    QString *parseMailbox( mailimf_mailbox *box );
+    QString parseMailboxList( mailimf_mailbox_list *list );
-    QString *parseGroup( mailimf_group *group );
+    QString parseMailbox( mailimf_mailbox *box );
-    QString *parseAddressList( mailimf_address_list *list );
+    QString parseGroup( mailimf_group *group );
-    QString *parseDateTime( mailimf_date_time *date );
+    QString parseAddressList( mailimf_address_list *list );
+    QString parseDateTime( mailimf_date_time *date );
    POP3account *account;
    mailpop3 *m_pop3;
 };
 #endif