REV: do not sort by key in Artist.update

tacaswell · tacaswell · commit bba3ae28a0f5 · 2015-12-01T14:39:04.000-05:00
restore old behavior
diff --git a/doc/api/api_changes/2015-12-01-TAC.rst b/doc/api/api_changes/2015-12-01-TAC.rst
@@ -6,13 +6,14 @@ The methods `matplotlib.artist.Artist.set`,
 now use a common codepath to look up how to update the given artist
 properties (either using the setter methods or an attribute/property).
 
-The behavior of `matplotlib.Artist.update` is slightly changed to now
-sort by key name and returns a list of the returned values from the
-setter methods to avoid changing the API of
-`matplotlib.Artist.set` and `matplotlib.artist.setp`.
+The behavior of `matplotlib.Artist.update` is slightly changed to
+returna a list of the returned values from the setter methods to avoid
+changing the API of `matplotlib.Artist.set` and
+`matplotlib.artist.setp`.
 
 The keys passed into `matplotlib.Artist.update` are now converted to
-all lower case before being processed, to match the behavior of
+lower case before being processed to match the behavior of
 `matplotlib.Artist.set` and `matplotlib.artist.setp`.  This should not
 break any user code because there are no set methods with capitals in
-the names, however going forward this puts a constraint naming properties.
+the names, however going forward this puts a constraint naming
+properties.
diff --git a/lib/matplotlib/artist.py b/lib/matplotlib/artist.py
@@ -872,7 +872,7 @@ def _update_property(self, k, v):
         self.eventson = False
         try:
             ret = [_update_property(self, k, v)
-                   for k, v in sorted(props.items(), reverse=True)]
+                   for k, v in props.items()]
         finally:
             self.eventson = store
 
