Refactored by Sourcery

Author: SourceryAI

examples/echo.py

@@ -75,8 +75,6 @@ def on_audio_receive_frame(self, fid, pcm, sample_count,
         sys.stdout.flush()
         bret = self.audio_send_frame(fid, pcm, sample_count,
                                      channels, sampling_rate)
-        if bret is False:
-            pass
 
     def on_video_receive_frame(self, fid, width, height, frame):
         # print('video frame: %d, %d, %d, ' % (fid, width, height))
@@ -85,7 +83,6 @@ def on_video_receive_frame(self, fid, width, height, frame):
         bret = self.video_send_frame(fid, width, height, frame)
         if bret is False:
             print('video send frame error.')
-            pass
 
     def witerate(self):
         self.iterate()

examples/phone.py

@@ -130,9 +130,8 @@ def on_end(self, idx):
         self.stop = True
         if self.ae_thread:
             self.ae_thread.join()
-        if self.call_type == self.TypeVideo:
-            if self.ve_thread:
-                self.ve_thread.join()
+        if self.call_type == self.TypeVideo and self.ve_thread:
+            self.ve_thread.join()
 
         self.kill_transmission(idx)
         print("Call ended")
@@ -261,9 +260,6 @@ def loop(self):
 
                 if readable:
                     args = sys.stdin.readline().strip().split()
-                    if not args:
-                        pass
-
                     if args[0] == "add":
                         try:
                             self.friend_add(args[1], "Hi")
@@ -272,8 +268,6 @@ def loop(self):
                         print('Friend added')
                     elif args[0] == "msg":
                         try:
-                            if len(args) <= 2:
-                                pass
                             friend_number = int(args[1])
                             msg = ' '.join(args[2:])
                             self.friend_send_message(friend_number,

tests/tests.py

@@ -115,7 +115,7 @@ def loop(self, n):
         t:iteration_interval
         """
         interval = self.bob.iteration_interval()
-        for i in range(n):
+        for _ in range(n):
             self.alice.iterate()
             self.bob.iterate()
             sleep(interval / 2000.0)
@@ -127,10 +127,7 @@ def loop_until_connected(self):
         """
 
         def on_self_connection_status(self, status):
-            if status != Tox.CONNECTION_NONE:
-                self.mycon_status = True
-            else:
-                self.mycon_status = False
+            self.mycon_status = True if status != Tox.CONNECTION_NONE else False
             assert self.self_get_connection_status() == status
 
         self.alice.mycon_status = False
@@ -160,7 +157,7 @@ def wait_callbacks(self, obj, attrs):
         count = 0
         THRESHOLD = 400
 
-        while not all([getattr(obj, attr) for attr in attrs]):
+        while not all(getattr(obj, attr) for attr in attrs):
             self.loop(50)
             if count >= THRESHOLD:
                 return False
@@ -731,7 +728,6 @@ def on_file_recv_control2(self, fid, file_number, control):
                 CONTEXT['START'] = True
             elif control == Tox.FILE_CONTROL_CANCEL:
                 self.completed = True
-                pass
 
         def on_file_chunk_request(self, fid, file_number, position, length):
             if length == 0:
@@ -759,8 +755,7 @@ def on_file_chunk_request(self, fid, file_number, position, length):
         BobTox.on_file_chunk_request = Tox.on_file_chunk_request
 
 if __name__ == '__main__':
-    methods = set([x for x in dir(Tox)
-                  if not x[0].isupper() and not x[0] == '_'])
+    methods = {x for x in dir(Tox) if not x[0].isupper() and x[0] != '_'}
     docs = "".join([getattr(ToxTest, x).__doc__ for x in dir(ToxTest)
                     if getattr(ToxTest, x).__doc__ is not None])