Strip comments from color schemes

This has been a problem.  Because xml.parser.expat doesn't seem to be
able to handle comments like this: `<!-- comments --- -->`, it is
probably best to strip them out.

Another common problem is that people put comments before the encoding
declaration.  This will address that as well.  It is easier to just
clean it up than educate people.

ref: #24 and #22

Author: facelessuser

lib/color_scheme_matcher.py

@@ -29,11 +29,7 @@
 from .rgba import RGBA
 from os import path
 from collections import namedtuple
-ST3 = int(sublime.version()) >= 3000
-if not ST3:
-    from plistlib import readPlist
-else:
-    from plistlib import readPlistFromBytes
+from plistlib import readPlistFromBytes
 
 
 class SchemeColors(
@@ -72,14 +68,14 @@ def __init__(self, scheme_file, ignore_gutter=False, track_dark_background=False
             color_filter = self.filter
         self.color_scheme = path.normpath(scheme_file)
         self.scheme_file = path.basename(self.color_scheme)
-        if ST3:
-            self.plist_file = color_filter(
-                readPlistFromBytes(sublime.load_binary_resource(sublime_format_path(self.color_scheme)))
-            )
-        else:
-            self.plist_file = color_filter(
-                readPlist(sublime.packages_path() + self.color_scheme.replace('Packages', ''))
+        self.plist_file = color_filter(
+            readPlistFromBytes(
+                re.sub(
+                    br"(\r?\n?\s*)*<!--[\s\S]*?-->(\s*\r?\n?)*", b'',
+                    sublime.load_binary_resource(sublime_format_path(self.color_scheme))
+                )
             )
+        )
         self.scheme_file = scheme_file
         self.ignore_gutter = ignore_gutter
         self.track_dark_background = track_dark_background