--- a/gmail-sentinel.conf.sample
+++ b/gmail-sentinel.conf.sample
@@ -20,7 +20,10 @@ browser = /usr/bin/firefox
 
 [plugins]
 path = /home/rodrigo/.gsentinel.d/plugins
-list = ""
+list = dummy
+
+[plugin-DummyPlugin]
+key = value
 
 [plugin-NotifyPlugin]
 iconpath = /usr/local/pshare/gmail-sentinel/media/

--- a/gmail-sentinel.py
+++ b/gmail-sentinel.py
@@ -29,7 +29,6 @@ popups.
 
 import pygtk
 pygtk.require('2.0')
-import time
 import gtk, gobject
 import locale, gettext
 import sys, os
@@ -37,12 +36,13 @@ import logging
 from os.path import dirname, pardir
 from gettext import gettext as _
 from ConfigParser import ConfigParser
-from gmailatom import GmailAtom
+from gsentinel.gmailatom import GmailAtom
 from gsentinel.plugin import PluginManager
 
 APP_NAME = "Gmail-Sentinel"
 APP_VERSION = "0.2_beta2"
-APP_COPYRIGHT = "(C)2008 Rodrigo Lazo.\n" + _("GoogleMail and Gmail are trademarks of Google, Inc.")
+APP_COPYRIGHT = "(C)2008 Rodrigo Lazo.\n" + \
+    _("GoogleMail and Gmail are trademarks of Google, Inc.")
 APP_DESCRIPTION = _('GoogleMail Sentinel')
 APP_WEBSITE = 'http://code.google.com/p/gmail-sentinel/'
 
@@ -102,7 +102,7 @@ class GmailStatusIcon(gtk.StatusIcon):
         dialog.set_comments(APP_DESCRIPTION)
         dialog.set_website(APP_WEBSITE)
         dialog.set_logo(gtk.gdk.pixbuf_new_from_file(self.gmail_icon))
-	dialog.set_translator_credits(_("translator-credits"))
+        dialog.set_translator_credits(_("translator-credits"))
         dialog.run()
         dialog.destroy()
 
@@ -119,34 +119,35 @@ class Gnotify:
         """Initializer. Creates the status icon and opens the
         connection."""
         self.config = ConfigParser()
-	self.plugin_manager = PluginManager()
+        self.read_msgs = list()
+        self.plugin_manager = PluginManager()
         self.connection = None
         self.unread_msg_count = 0
 
-	logging.info("[Main] Loading configuration file")
+        logging.info("[Main] Loading configuration file")
         if not self.load_configuration():
             logging.critical("[Main] Configuration file does not exist.")
             sys.exit(1)
 
-	# plugin system initialization
-	logging.info("[Main] Starting PluginManager")
-	self.plugin_manager.initialize(self.config.get('plugins', 'path'), 
-				       self.config.get('plugins', 'list'))
-	logging.info("[Main] Creating StatusIcon")
+        logging.info("[Main] Creating StatusIcon")
         self.status_icon = GmailStatusIcon(self.config.get('media', 'path'))
-	
+        
+        logging.info("[Main] Starting PluginManager")
+        self.plugin_manager.initialize(self.config)
+
         # Make sure the status icon shows up
         while gtk.events_pending():
             gtk.main_iteration(True)
-	logging.info("[Main] First run. Checking for new mail")
+        logging.info("[Main] First run. Checking for new mail")
         if self.connect() :
             self.check_inbox()
         # The multiplier is to simplify the configuration
         # file. Converts the miliseconds into seconds
-	logging.info("[Main] Registering interval function")
+        logging.info("[Main] Registering interval function")
         interval = int(self.config.get('gmail', 'checkinterval'))*1000
         self.main_timer = gobject.timeout_add(interval, self.check_inbox)
-	logging.info("[Main] Sleeping interval %d miliseconds. Sleeping now.", interval)
+        logging.info("[Main] Sleeping interval %d miliseconds. Sleeping now.",
+                     interval)
 
     def load_configuration (self):
         """Loads the configuration. If file does not exist returns
@@ -169,71 +170,65 @@ class Gnotify:
                 self.connection = GmailAtom(self.config.get('gmail','username'),
                                             self.config.get('gmail','password'))
             except:
-                logging.error("[Main] Login failed, will retry later")
+                logging.exception("[Main] Login failed, will retry later")
                 self.status_icon.set_tooltip(_("Connection failed"))
                 return False
-	if self.connection.refreshInfo():
-	    logging.info("[Main] Connection open")
-	    self.status_icon.set_tooltip(_("Connected"))
-	    return True
-	else:
-	    logging.error("[Main] Connection failed")
-	    self.status_icon.set_tooltip(_("Not connected"))
-	    return False
+        if self.connection.refreshInfo():
+            logging.info("[Main] Connection open")
+            self.status_icon.set_tooltip(_("Connected"))
+            return True
+        else:
+            logging.error("[Main] Connection failed")
+            self.status_icon.set_tooltip(_("Not connected"))
+            return False
 
     def check_inbox(self):
         """Checks for new mail."""
         if self.connection is None or self.connect() is False:
             # Try again latter
             return True
-	logging.info("[Main] Checking for new mail.")
+        logging.info("[Main] Checking for new mail.")
         while gtk.events_pending():
             gtk.main_iteration( True )
 
         if not self.has_new_mail():
-	    logging.info("[Main] No new messages")
-	    self.plugin_manager.emit_signal('no_new_messages')
+            logging.info("[Main] No new messages")
+            self.plugin_manager.emit_signal('no_new_messages')
             # Check again later
-            self.unread_msg_count > 0 or \
+            len(self.read_msgs) == 0 or \
                 self.status_icon.set_from_file(self.status_icon.inactive_icon)
             return True
         self.status_icon.set_from_file(self.status_icon.active_icon)
         count = self.connection.getUnreadMsgCount()
-	logging.info("[Main] %s New messages found", count)
-        last_message = self.get_last_message()
-	self.plugin_manager.emit_signal('new_messages', messages=self.connection.getMsgList())
+        logging.info("[Main] %s New messages found", count)
+        self.plugin_manager.emit_signal('new_messages',
+                                        messages=self.connection.getMsgList())
         return True
 
     def has_new_mail (self):
         """Checks if new mail has arrived."""
+        # TODO: new mail != read mail != no mail
         try:
-            # Hackerish?
             self.connection.refreshInfo()
-            current_msg_count = self.connection.getUnreadMsgCount()
-            new_mail = current_msg_count > 0 and \
-                current_msg_count != self.unread_msg_count
-            self.unread_msg_count = current_msg_count
-            return new_mail
-        except:
-	    logging.error("[Main] getUnreadMsgCount() failed, will try again later")
-            self.unread_msg_count = 0
+            msgs = self.connection.getMsgList()
+            # are all the messages already seen?
+            set1 = set(x["id"] for x in self.read_msgs)
+            same_messages = set1.issuperset(x["id"] for x in msgs)
+            if not same_messages:
+                self.read_msgs = msgs
+            return not same_messages
+        except Exception, ex:
+            logging.exception("[Main] getUnreadMsgCount() failed, will try again later")
             return False
 
-    def get_last_message (self):
-        """ Returns the sender, subject and snippet of the last
-        message retrieved."""
-        alist = self.connection.getMsgList()
-        return {'sender' : alist[0]['author_name'],
-                'subject' : alist[0]['title'],
-                'snippet' : alist[0]['summary'][:40] + "..."}
-
     def main (self):
         """ Mailoop"""
         gtk.main()
 
 if __name__ == "__main__":
     # Path definition
-    SHARE_PATH = os.path.abspath(os.path.join(dirname(__file__), pardir, pardir, 'share'))
+    SHARE_PATH = os.path.abspath(os.path.join(dirname(__file__), 
+                                              pardir, pardir, 'share'))
     LOCALE_PATH = os.path.join(SHARE_PATH, 'locale')
 
     # L10n initialization
@@ -243,7 +238,7 @@ if __name__ == "__main__":
 
     # Loggging configuration
     logging.basicConfig(level=logging.INFO,
-			format='%(asctime)s %(levelname)s %(message)s')
+                        format='%(asctime)s %(levelname)s %(message)s')
 
     # app itself
     notifier = Gnotify()

--- a/gmailatom.py
+++ /dev/null
@@ -1,189 +0,0 @@
-#!/usr/bin/python
-# -*- coding: utf-8 -*-
-
-# gmailatom 0.0.2
-#
-# by Juan Grande <juan.grande@gmail.com>
-# modified by Rodrigo Lazo <rlazo.paz@gmail.com>
-
-"""
-Gmailatom.
-
-Helps connection to Gmail using the atom interface.
-
-HOW TO USE:
-1) Create an instance of 'GmailAtom' class. The two arguments
-   its constructor take are the username (including '@gmail.com')
-   and the password.
-2) To retrieve the account status call 'refreshInfo()'.
-3) To get the unread messages count call 'getUnreadMsgCount()'.
-   You MUST call 'refreshInfo()' at least one time before using
-   this method or it will return zero.
-4) To get specific information about an unread email you must
-   call the corresponding getter method passing to it the number
-   of the message. The number zero represents the newest one.
-   You MUST call 'refreshInfo()' at least one time before calling getMsgList
-
-   The getMsgList returns a list of Messages (dictionary-like) with
-   the following data:
-       id
-       title
-       summary
-       author_name
-       author_email
-       link
-"""
-
-from xml.sax.handler import ContentHandler
-from xml import sax
-import urllib2
-
-# Because Message acts like a struct we only need to inheret from dict
-class Message (dict):
-    """Message struct"""
-    def __init__ (self):
-        self.update( {'id' : "",
-                      'title' : "",
-                      'summary' : "",
-                      'author_name' : "",
-                      'author_email' : "",
-                      'link' : "" } )
-
-    def __cmp__ (self, msg):
-        """Messages are compared by id"""
-        return cmp(self['id'], msg['id'])
-
-
-# Sax XML Handler
-class MailHandler(ContentHandler):
-    """Sax XML Handler for gmail atom interface"""
-
-    # Tags
-    TAG_FEED = "feed"
-    TAG_FULLCOUNT = "fullcount"
-    TAG_ENTRY = "entry"
-    TAG_TITLE = "title"
-    TAG_SUMMARY = "summary"
-    TAG_LINK = "link"
-    TAG_ID = "id"
-    TAG_AUTHOR = "author"
-    TAG_NAME = "name"
-    TAG_EMAIL = "email"
-
-    # Path the information
-    PATH_FULLCOUNT = [ TAG_FEED, TAG_FULLCOUNT ]
-    PATH_ENTRY = [ TAG_FEED, TAG_ENTRY ]
-    PATH_LINK = PATH_ENTRY + [ TAG_LINK ]
-    PATH_ID = PATH_ENTRY + [ TAG_ID ]
-    PATH_TITLE =  PATH_ENTRY + [ TAG_TITLE ]
-    PATH_SUMMARY = PATH_ENTRY + [ TAG_SUMMARY ]
-    PATH_AUTHOR = PATH_ENTRY + [ TAG_AUTHOR ]
-    PATH_AUTHOR_NAME = PATH_AUTHOR + [ TAG_NAME ]
-    PATH_AUTHOR_EMAIL = PATH_AUTHOR + [ TAG_EMAIL ]
-
-    def __init__(self):
-        ContentHandler.__init__(self)
-        self.entries = list()
-        self.actual = list()
-        self.mail_count = "0"
-        self.current_msg = None
-
-    def startDocument (self):
-        """Document initialization"""
-        del self.entries[:]
-        del self.actual[:]
-        self.mail_count = "0"
-        self.current_msg = None
-
-    def startElement( self, name, attrs):
-        # update actual path
-        self.actual.append(name)
-
-        # Message link
-        if self.actual == self.PATH_LINK:
-            self.current_msg['link'] = attrs['href']
-
-        # add a new email to the list
-        if name == "entry":
-            self.current_msg = Message()
-
-    def endElement( self, name):
-        # update actual path
-        if name == "entry":
-            self.entries.append(self.current_msg.copy())
-            self.current_msg.clear()
-        self.actual.pop()
-
-    def characters( self, content):
-        # New messages count
-        if (self.actual == self.PATH_FULLCOUNT):
-            self.mail_count = content
-
-        # Message title
-        if (self.actual == self.PATH_TITLE):
-            self.current_msg['title'] += content
-
-        # Message summary
-        if (self.actual == self.PATH_SUMMARY):
-            self.current_msg['summary'] += content
-
-        # Message id
-        if self.actual == self.PATH_ID:
-            self.current_msg['id'] += content
-
-        # Message author name
-        if (self.actual == self.PATH_AUTHOR_NAME):
-            self.current_msg['author_name'] += content
-
-        # Message author email
-        if (self.actual == self.PATH_AUTHOR_EMAIL):
-            self.current_msg['author_email'] += content
-
-    def getUnreadMsgCount(self):
-        return int(self.mail_count)
-
-# The mail class
-class GmailAtom:
-    """Gmail's atom interface proxy"""
-
-    realm = "New mail feed"
-    host = "https://mail.google.com"
-    url = host + "/mail/feed/atom"
-
-    def __init__(self, user, pswd):
-        self.mh = MailHandler()
-        # initialize authorization handler
-        auth_handler = urllib2.HTTPBasicAuthHandler()
-        auth_handler.add_password( self.realm, self.host, user, pswd)
-        opener = urllib2.build_opener(auth_handler)
-        urllib2.install_opener(opener)
-
-    def sendRequest(self):
-        """Opens a conection with the remote server"""
-        del self.mh.entries[:]
-	return urllib2.urlopen(self.url)
-
-    def refreshInfo(self):
-        """Requests information from the remote server and parses the
-        information retrieved"""
-	try:
-	    sax.parseString( self.sendRequest().read(), self.mh)
-	    return True
-	except urllib2.URLError:
-	    return False
-
-    def getMsgList (self):
-        """Returns a list containing the new messages or empty. It
-        will replace all the getMsg* functions in the next
-        release. Every entry on the list acts like a dictionary:
-
-        old api
-        getMsgSummary(1) #Get the summary of the first message
-
-        new api
-        l = getMsgList()
-        l[0]['summary']"""
-        return self.mh.entries
-
-    def getUnreadMsgCount(self):
-        return self.mh.getUnreadMsgCount()

--- a/gsentinel/__init__.py
+++ b/gsentinel/__init__.py
@@ -13,3 +13,22 @@
 # You should have received a copy of the GNU General Public License
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+# Because Message acts like a struct we only need to inheret from dict
+class Message (dict):
+    """Message struct"""
+    def __init__ (self):
+        self.update( {'id' : "",
+                      'title' : "",
+                      'summary' : "",
+                      'author_name' : "",
+                      'author_email' : "",
+                      'link' : ""} )
+
+    def __cmp__ (self, msg):
+        """Messages are compared by id"""
+        return cmp(self['id'], msg['id'])
+
+    def __hash__ (self):
+        """Hash method"""
+        return hash(self['id'])

--- /dev/null
+++ b/gsentinel/gmailatom.py
@@ -0,0 +1,175 @@
+#!/usr/bin/python
+# -*- coding: utf-8 -*-
+
+# gmailatom 0.0.2
+#
+# by Juan Grande <juan.grande@gmail.com>
+# modified by Rodrigo Lazo <rlazo.paz@gmail.com>
+
+"""
+Gmailatom.
+
+Helps connection to Gmail using the atom interface.
+
+HOW TO USE:
+1) Create an instance of 'GmailAtom' class. The two arguments
+   its constructor take are the username (including '@gmail.com')
+   and the password.
+2) To retrieve the account status call 'refreshInfo()'.
+3) To get the unread messages count call 'getUnreadMsgCount()'.
+   You MUST call 'refreshInfo()' at least one time before using
+   this method or it will return zero.
+4) To get specific information about an unread email you must
+   call the corresponding getter method passing to it the number
+   of the message. The number zero represents the newest one.
+   You MUST call 'refreshInfo()' at least one time before calling getMsgList
+
+   The getMsgList returns a list of Messages (dictionary-like) with
+   the following data:
+       id
+       title
+       summary
+       author_name
+       author_email
+       link
+"""
+
+from gsentinel import Message
+from xml.sax.handler import ContentHandler
+from datetime import datetime
+from xml import sax
+import urllib2
+
+# Sax XML Handler
+class MailHandler(ContentHandler):
+    """Sax XML Handler for gmail atom interface"""
+
+    # Tags
+    TAG_FEED = "feed"
+    TAG_FULLCOUNT = "fullcount"
+    TAG_ENTRY = "entry"
+    TAG_TITLE = "title"
+    TAG_SUMMARY = "summary"
+    TAG_LINK = "link"
+    TAG_ID = "id"
+    TAG_AUTHOR = "author"
+    TAG_NAME = "name"
+    TAG_EMAIL = "email"
+
+    # Path the information
+    PATH_FULLCOUNT = [ TAG_FEED, TAG_FULLCOUNT ]
+    PATH_ENTRY = [ TAG_FEED, TAG_ENTRY ]
+    PATH_LINK = PATH_ENTRY + [ TAG_LINK ]
+    PATH_ID = PATH_ENTRY + [ TAG_ID ]
+    PATH_TITLE =  PATH_ENTRY + [ TAG_TITLE ]
+    PATH_SUMMARY = PATH_ENTRY + [ TAG_SUMMARY ]
+    PATH_AUTHOR = PATH_ENTRY + [ TAG_AUTHOR ]
+    PATH_AUTHOR_NAME = PATH_AUTHOR + [ TAG_NAME ]
+    PATH_AUTHOR_EMAIL = PATH_AUTHOR + [ TAG_EMAIL ]
+
+    def __init__(self):
+        ContentHandler.__init__(self)
+        self.entries = list()
+        self.actual = list()
+        self.mail_count = "0"
+        self.current_msg = None
+
+    def startDocument (self):
+        """Document initialization"""
+        del self.entries[:]
+        del self.actual[:]
+        self.mail_count = "0"
+        self.current_msg = None
+
+    def startElement( self, name, attrs):
+        # update actual path
+        self.actual.append(name)
+
+        # Message link
+        if self.actual == self.PATH_LINK:
+            self.current_msg['link'] = attrs['href']
+
+        # add a new email to the list
+        if name == "entry":
+            self.current_msg = Message()
+
+    def endElement( self, name):
+        # update actual path
+        if name == "entry":
+            self.entries.append(self.current_msg.copy())
+            self.current_msg.clear()
+        self.actual.pop()
+
+    def characters( self, content):
+        # New messages count
+        if (self.actual == self.PATH_FULLCOUNT):
+            self.mail_count = content
+
+        # Message title
+        if (self.actual == self.PATH_TITLE):
+            self.current_msg['title'] += content
+
+        # Message summary
+        if (self.actual == self.PATH_SUMMARY):
+            self.current_msg['summary'] += content
+
+        # Message id
+        if self.actual == self.PATH_ID:
+            self.current_msg['id'] += content
+
+        # Message author name
+        if (self.actual == self.PATH_AUTHOR_NAME):
+            self.current_msg['author_name'] += content
+
+        # Message author email
+        if (self.actual == self.PATH_AUTHOR_EMAIL):
+            self.current_msg['author_email'] += content
+
+    def getUnreadMsgCount(self):
+        return int(self.mail_count)
+
+# The mail class
+class GmailAtom:
+    """Gmail's atom interface proxy"""
+
+    realm = "New mail feed"
+    host = "https://mail.google.com"
+    url = host + "/mail/feed/atom"
+
+    def __init__(self, user, pswd):
+        self.mh = MailHandler()
+        # initialize authorization handler
+        auth_handler = urllib2.HTTPBasicAuthHandler()
+        auth_handler.add_password( self.realm, self.host, user, pswd)
+        opener = urllib2.build_opener(auth_handler)
+        urllib2.install_opener(opener)
+
+    def sendRequest(self):
+        """Opens a conection with the remote server"""
+        del self.mh.entries[:]
+	return urllib2.urlopen(self.url)
+
+    def refreshInfo(self):
+        """Requests information from the remote server and parses the
+        information retrieved"""
+	try:
+	    sax.parseString( self.sendRequest().read(), self.mh)
+	    return True
+	except urllib2.URLError:
+	    return False
+
+    def getMsgList (self):
+        """Returns a list containing the new messages or empty. It
+        will replace all the getMsg* functions in the next
+        release. Every entry on the list acts like a dictionary:
+
+        old api
+        getMsgSummary(1) #Get the summary of the first message
+
+        new api
+        l = getMsgList()
+        l[0]['summary']"""
+        return self.mh.entries
+
+    def getUnreadMsgCount(self):
+        return self.mh.getUnreadMsgCount()