On 11/19/2010 10:13 PM, Sascha Silbe wrote:
Reviewed-by: James Cameron<qu...@laptop.org>
CC: Aleksey Lim<alsr...@member.fsf.org>
Signed-off-by: Sascha Silbe<sascha-...@silbe.org>

diff --git a/src/jarabe/desktop/activitieslist.py 
b/src/jarabe/desktop/activitieslist.py
index be60c4c..c25e523 100644
--- a/src/jarabe/desktop/activitieslist.py
+++ b/src/jarabe/desktop/activitieslist.py
@@ -420,8 +420,8 @@ class ActivityListPalette(ActivityPalette):
          menu_item.show()

          if not os.access(activity_info.get_path(), os.W_OK) or \
-            registry.is_activity_protected(self._bundle_id):
-                menu_item.props.sensitive = False
+           registry.is_activity_protected(self._bundle_id):
+            menu_item.props.sensitive = False

      def __destroy_cb(self, palette):
          self.disconnect(self._activity_changed_sid)

Shouldn't

registry.is_activity_protected(self._bundle_id):

be inlined by 8?

Would at least be better readable.
Simon
_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to