pep8 & unused variable

synapse/push/__init__.py

@@ -26,6 +26,7 @@ import random
 
 logger = logging.getLogger(__name__)
 
+
 # Pushers could now be moved to pull out of the event_actions table instead
 # of listening on the event stream: this would avoid them having to run the
 # rules again.

synapse/push/action_generator.py

@@ -35,8 +35,6 @@ class ActionGenerator:
 
     @defer.inlineCallbacks
     def handle_event(self, event):
-        users = yield self.store.get_users_in_room(event['room_id'])
-
         bulk_evaluator = yield bulk_push_rule_evaluator.evaluator_for_room_id(
             event['room_id'], self.hs, self.store
         )

synapse/push/bulk_push_rule_evaluator.py

@@ -54,7 +54,7 @@ def evaluator_for_room_id(room_id, hs, store):
             display_names[ev.state_key] = ev.content.get("displayname")
 
     defer.returnValue(BulkPushRuleEvaluator(
-            room_id, rules_by_user, display_names, users
+        room_id, rules_by_user, display_names, users
     ))
 
 
@@ -104,4 +104,4 @@ class BulkPushRuleEvaluator:
             matches &= PushRuleEvaluator._event_fulfills_condition(
                 event, cond, display_name, room_member_count, profile_tag
             )
-        return matches
+        return matches