From bdb763d9cff75eec4bb44d23bfc2ef6fa4871bcc Mon Sep 17 00:00:00 2001
From: Maximilian Hils <git@maximilianhils.com>
Date: Mon, 8 Feb 2016 04:19:25 +0100
Subject: make stateobject simpler and stricter

---
 libmproxy/stateobject.py | 47 +++++++++++++++++++++++++----------------------
 1 file changed, 25 insertions(+), 22 deletions(-)

(limited to 'libmproxy/stateobject.py')

diff --git a/libmproxy/stateobject.py b/libmproxy/stateobject.py
index 6f15ae7a..9600ab09 100644
--- a/libmproxy/stateobject.py
+++ b/libmproxy/stateobject.py
@@ -1,26 +1,25 @@
 from __future__ import absolute_import
+from netlib.utils import Serializable
 
 
-class StateObject(object):
-
+class StateObject(Serializable):
     """
-        An object with serializable state.
+    An object with serializable state.
 
-        State attributes can either be serializable types(str, tuple, bool, ...)
-        or StateObject instances themselves.
+    State attributes can either be serializable types(str, tuple, bool, ...)
+    or StateObject instances themselves.
     """
-    # An attribute-name -> class-or-type dict containing all attributes that
-    # should be serialized. If the attribute is a class, it must implement the
-    # StateObject protocol.
-    _stateobject_attributes = None
 
-    def from_state(self, state):
-        raise NotImplementedError()
+    _stateobject_attributes = None
+    """
+    An attribute-name -> class-or-type dict containing all attributes that
+    should be serialized. If the attribute is a class, it must implement the
+    Serializable protocol.
+    """
 
     def get_state(self):
         """
-            Retrieve object state. If short is true, return an abbreviated
-            format with long data elided.
+        Retrieve object state.
         """
         state = {}
         for attr, cls in self._stateobject_attributes.iteritems():
@@ -31,18 +30,22 @@ class StateObject(object):
                 state[attr] = val
         return state
 
-    def load_state(self, state):
+    def set_state(self, state):
         """
-            Load object state from data returned by a get_state call.
+        Load object state from data returned by a get_state call.
         """
+        state = state.copy()
         for attr, cls in self._stateobject_attributes.iteritems():
-            if state.get(attr, None) is None:
-                setattr(self, attr, None)
+            if state.get(attr) is None:
+                setattr(self, attr, state.pop(attr))
             else:
                 curr = getattr(self, attr)
-                if hasattr(curr, "load_state"):
-                    curr.load_state(state[attr])
+                if hasattr(curr, "set_state"):
+                    curr.set_state(state.pop(attr))
                 elif hasattr(cls, "from_state"):
-                    setattr(self, attr, cls.from_state(state[attr]))
-                else:
-                    setattr(self, attr, cls(state[attr]))
+                    obj = cls.from_state(state.pop(attr))
+                    setattr(self, attr, obj)
+                else:  # primitive types such as int, str, ...
+                    setattr(self, attr, cls(state.pop(attr)))
+        if state:
+            raise RuntimeWarning("Unexpected State in __setstate__: {}".format(state))
-- 
cgit v1.2.3