-resolved that not all newsgroups where listed

-resolved a memory leak
-newsgroups list enhanced for usage with wildcards

2 files changed, 17 insertions, 4 deletions

diff --git a/noncore/net/mail/libmailwrapper/nntpwrapper.cpp b/noncore/net/mail/libmailwrapper/nntpwrapper.cpp
index a766a59..5a8c224 100644
--- a/noncore/net/mail/libmailwrapper/nntpwrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/nntpwrapper.cpp
@@ -204,3 +204,9 @@ QList<Folder>* NNTPwrapper::listFolders() {
 
-QStringList NNTPwrapper::listAllNewsgroups() {
+/* we made this method in raw nntp access of etpan and not via generic interface
+ * 'cause in that case there will be doubled copy operations. eg. the etpan would
+ * copy that stuff into its own structures and we must copy it into useable c++ 
+ * structures for our frontend. this would not make sense, so it is better to reimplement
+ * the stuff from generic interface of etpan but copy it direct to qt classes.
+ */
+QStringList NNTPwrapper::listAllNewsgroups(const QString&mask) {
     login();
@@ -214,3 +220,10 @@ QStringList NNTPwrapper::listAllNewsgroups() {
         newsnntp * news =  ( (  nntp_session_state_data * )session->sess_data )->nntp_session;
-        int err = newsnntp_list_newsgroups(news, NULL, &result);
+        int err = NEWSNNTP_NO_ERROR;
+        if (mask.isEmpty()) {
+            err = newsnntp_list(news, &result);
+        } else {
+            /* taken from generic wrapper of etpan */
+            QString nmask = mask+".*";
+            err = newsnntp_list_active(news, nmask.latin1(), &result);
+        }
         if ( err == NEWSNNTP_NO_ERROR && result) {
@@ -224,3 +237,3 @@ QStringList NNTPwrapper::listAllNewsgroups() {
     if (result) {
-        clist_free(result);
+        newsnntp_list_free(result);
     }
diff --git a/noncore/net/mail/libmailwrapper/nntpwrapper.h b/noncore/net/mail/libmailwrapper/nntpwrapper.h
index 4d03754..d51c955 100644
--- a/noncore/net/mail/libmailwrapper/nntpwrapper.h
+++ b/noncore/net/mail/libmailwrapper/nntpwrapper.h
@@ -27,3 +27,3 @@ public:
     virtual void statusFolder(folderStat&target_stat,const QString & mailbox="INBOX");
-    QStringList listAllNewsgroups();
+    QStringList listAllNewsgroups(const QString&mask = QString::null);
     virtual void deleteMail(const RecMail&mail);