Updated README and added _BaseNote

Author: Kenny2github

README.rst

@@ -23,5 +23,5 @@ Assuming your .chart file is named "notes.chart"...
     >>> import chparse
     >>> with open('notes.chart') as chartfile:
     ...     chart = chparse.load(chartfile)
-    >>> chart.instruments[chparse.EXPERT][chparse.GUITAR].notes[0]
+    >>> chart.instruments[chparse.EXPERT][chparse.GUITAR][0]
     <Note: 0 = N 3 0 (<Flags.NONE: 0>)>

chparse/__init__.py

@@ -24,7 +24,7 @@
     >>> import chparse
     >>> with open('notes.chart') as chartfile:
     ...     chart = chparse.load(chartfile)
-    >>> chart.instruments[chparse.EXPERT][chparse.GUITAR].notes[0]
+    >>> chart.instruments[chparse.EXPERT][chparse.GUITAR][0]
     <Note: 0 = N 3 0 (<Flags.NONE: 0>)>
 """
 from .parse import load, dump

chparse/note.py

@@ -1,9 +1,42 @@
 """Contains the Note class."""
 from . import flags as _flags
 
-class Note(object):
-    """Represents a single note - i.e. 0 = X 0 0"""
+class _BaseNote(object):
+    """Represents anything resembling a note."""
     time = 0
+
+    def __cmp__(self, other):
+        if not isinstance(other, _BaseNote):
+            raise TypeError('Cannot compare Note with {.__name__}'
+                            .format(type(other)))
+        return (-1
+                if self.time < other.time
+                else (1
+                      if self.time > other.time
+                      else 0
+                )
+        )
+
+    def __lt__(self, other):
+        return self.__cmp__(other) < 0
+
+    def __le__(self, other):
+        return self.__cmp__(other) <= 0
+
+    def __eq__(self, other):
+        return self.__cmp__(other) == 0
+
+    def __ne__(self, other):
+        return self.__cmp__(other) != 0
+
+    def __gt__(self, other):
+        return self.__cmp__(other) > 0
+
+    def __ge__(self, other):
+        return self.__cmp__(other) >= 0
+
+class Note(_BaseNote):
+    """Represents a single note - i.e. 0 = X 0 0"""
     kind = _flags.NOTE
     fret = 0
     length = 0
@@ -55,36 +88,6 @@ def __str__(self):
         result += '\n' + result2
         return result
 
-    def __cmp__(self, other):
-        if not isinstance(other, (Note, Event, SyncEvent)):
-            raise TypeError('Cannot compare Note with {.__name__}'
-                            .format(type(other)))
-        return (-1
-                if self.time < other.time
-                else (1
-                      if self.time > other.time
-                      else 0
-                )
-        )
-
-    def __lt__(self, other):
-        return self.__cmp__(other) < 0
-
-    def __le__(self, other):
-        return self.__cmp__(other) <= 0
-
-    def __eq__(self, other):
-        return self.__cmp__(other) == 0
-
-    def __ne__(self, other):
-        return self.__cmp__(other) != 0
-
-    def __gt__(self, other):
-        return self.__cmp__(other) > 0
-
-    def __ge__(self, other):
-        return self.__cmp__(other) >= 0
-
     @property
     def is_tap(self):
         """Return whether this Note is a tap note."""
@@ -109,9 +112,8 @@ def is_forced(self):
             else (_flags.FORCED in self.flags)
         )
 
-class Event(object): #pylint: disable=too-few-public-methods
+class Event(_BaseNote): #pylint: disable=too-few-public-methods
     """Represents the special E note for events."""
-    time = 0
     event = ''
 
     def __init__(self, time, evt):
@@ -124,40 +126,8 @@ def __repr__(self):
     def __str__(self):
         return '  {} = E {}'.format(self.time, self.event)
 
-    def __cmp__(self, other):
-        if not isinstance(other, (Note, Event, SyncEvent)):
-            raise TypeError('Cannot compare Event with {.__name__}'
-                            .format(type(other)))
-        return (-1
-                if self.time < other.time
-                else (1
-                      if self.time > other.time
-                      else 0
-                )
-        )
-
-    def __lt__(self, other):
-        return self.__cmp__(other) < 0
-
-    def __le__(self, other):
-        return self.__cmp__(other) <= 0
-
-    def __eq__(self, other):
-        return self.__cmp__(other) == 0
-
-    def __ne__(self, other):
-        return self.__cmp__(other) != 0
-
-    def __gt__(self, other):
-        return self.__cmp__(other) > 0
-
-    def __ge__(self, other):
-        return self.__cmp__(other) >= 0
-
-
-class SyncEvent(object):
+class SyncEvent(_BaseNote):
     """Represents the special TS and B notes in the SyncTrack instrument."""
-    time = 0
     kind = _flags.BPM
     value = 0
 
@@ -170,33 +140,3 @@ def __repr__(self):
 
     def __str__(self):
         return '  {} = {} {}'.format(self.time, self.kind.value, self.value)
-
-    def __cmp__(self, other):
-        if not isinstance(other, (Note, Event, SyncEvent)):
-            raise TypeError('Cannot compare SyncEvent with {.__name__}'
-                            .format(type(other)))
-        return (-1
-                if self.time < other.time
-                else (1
-                      if self.time > other.time
-                      else 0
-                )
-        )
-
-    def __lt__(self, other):
-        return self.__cmp__(other) < 0
-
-    def __le__(self, other):
-        return self.__cmp__(other) <= 0
-
-    def __eq__(self, other):
-        return self.__cmp__(other) == 0
-
-    def __ne__(self, other):
-        return self.__cmp__(other) != 0
-
-    def __gt__(self, other):
-        return self.__cmp__(other) > 0
-
-    def __ge__(self, other):
-        return self.__cmp__(other) >= 0