From f1cb3dfa041a181402e242a7e4bc0f2eaf20be2a Mon Sep 17 00:00:00 2001
From: Cole Robinson <crobinso@redhat.com>
Date: Sat, 10 Aug 2013 15:46:13 -0400
Subject: [PATCH] Guest: Remove needless _OS_TYPES storage

---
 virtinst/guest.py | 21 ++++++++-------------
 1 file changed, 8 insertions(+), 13 deletions(-)

diff --git a/virtinst/guest.py b/virtinst/guest.py
index 2cfa456f7..037a201e2 100644
--- a/virtinst/guest.py
+++ b/virtinst/guest.py
@@ -41,11 +41,6 @@ from virtinst import osdict
 
 
 class Guest(XMLBuilder):
-
-    # OS Dictionary static variables and methods
-    _DEFAULTS = osdict.DEFAULTS
-    _OS_TYPES = osdict.OS_TYPES
-
     @staticmethod
     def pretty_os_list():
         """
@@ -62,7 +57,7 @@ class Guest(XMLBuilder):
         """
         @param filtervars: List of only variants we want to show by default
         """
-        vals = osdict.sort_helper(Guest._OS_TYPES)
+        vals = osdict.sort_helper(osdict.OS_TYPES)
         for t in vals[:]:
             if not Guest.list_os_variants(t, supported=supported,
                                           filtervars=filtervars):
@@ -79,7 +74,7 @@ class Guest(XMLBuilder):
         the sorted list have all fedora distros first
         @param filtervars: List of only variants we want to show by default
         """
-        vals = osdict.sort_helper(Guest._OS_TYPES[typ]["variants"],
+        vals = osdict.sort_helper(osdict.OS_TYPES[typ]["variants"],
                                   sortpref)
         ret = []
         for v in vals:
@@ -96,11 +91,11 @@ class Guest(XMLBuilder):
 
     @staticmethod
     def get_os_type_label(typ):
-        return Guest._OS_TYPES[typ]["label"]
+        return osdict.OS_TYPES[typ]["label"]
 
     @staticmethod
     def get_os_variant_label(typ, variant):
-        return Guest._OS_TYPES[typ]["variants"][variant]["label"]
+        return osdict.OS_TYPES[typ]["variants"][variant]["label"]
 
     @staticmethod
     def check_vm_collision(conn, name, do_remove):
@@ -245,7 +240,7 @@ class Guest(XMLBuilder):
         return self._os_type
     def set_os_type(self, val):
         val = val.lower()
-        if val in self._OS_TYPES:
+        if val in osdict.OS_TYPES:
             if self._os_type != val:
                 # Invalidate variant, since it may not apply to the new os type
                 self._os_type = val
@@ -261,7 +256,7 @@ class Guest(XMLBuilder):
         val = val.lower()
 
         if self.os_type:
-            if val in self._OS_TYPES[self.os_type]["variants"]:
+            if val in osdict.OS_TYPES[self.os_type]["variants"]:
                 self._os_variant = val
             else:
                 raise ValueError(_("OS variant '%(var)s' does not exist in "
@@ -270,8 +265,8 @@ class Guest(XMLBuilder):
         else:
             found = False
             for ostype in self.list_os_types():
-                if (val in self._OS_TYPES[ostype]["variants"] and
-                    not self._OS_TYPES[ostype]["variants"][val].get("skip")):
+                if (val in osdict.OS_TYPES[ostype]["variants"] and
+                    not osdict.OS_TYPES[ostype]["variants"][val].get("skip")):
                     logging.debug("Setting os type to '%s' for variant '%s'",
                                   ostype, val)
                     self.os_type = ostype