[Lldb-commits] [PATCH] D19510: Fix send and receive of ACK byte in test infrastructure for Python 3.5

[Adrian McCarthy via lldb-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL267562: Fix send and receive of ACK byte in test infrastructure for Python 3.5 (authored by amccarth).

Changed prior to commit:
  http://reviews.llvm.org/D19510?vs=54938&id=55005#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D19510

Files:
  lldb/trunk/packages/Python/lldbsuite/test_event/dotest_channels.py
  lldb/trunk/packages/Python/lldbsuite/test_event/formatter/__init__.py

Index: lldb/trunk/packages/Python/lldbsuite/test_event/dotest_channels.py
===================================================================
--- lldb/trunk/packages/Python/lldbsuite/test_event/dotest_channels.py
+++ lldb/trunk/packages/Python/lldbsuite/test_event/dotest_channels.py
@@ -59,8 +59,7 @@
         # the initiators of the socket to await this to ensure
         # that this end is up and running (and therefore already
         # into the async map).
-        ack_bytes = bytearray()
-        ack_bytes.append(chr(42))
+        ack_bytes = b'*'
         file_object.send(ack_bytes)
 
     def deserialize_payload(self):
Index: lldb/trunk/packages/Python/lldbsuite/test_event/formatter/__init__.py
===================================================================
--- lldb/trunk/packages/Python/lldbsuite/test_event/formatter/__init__.py
+++ lldb/trunk/packages/Python/lldbsuite/test_event/formatter/__init__.py
@@ -40,9 +40,6 @@
         self.cleanup_func = cleanup_func
 
 
-SOCKET_ACK_BYTE_VALUE = b'*'  # ASCII for chr(42)
-
-
 def create_results_formatter(config):
     """Sets up a test results formatter.
 
@@ -78,7 +75,7 @@
         # listener socket gets spun up; otherwise,
         # we lose the test result info.
         read_bytes = sock.recv(1)
-        if read_bytes is None or (len(read_bytes) < 1) or (read_bytes[0] != SOCKET_ACK_BYTE_VALUE):
+        if read_bytes is None or (len(read_bytes) < 1) or (read_bytes != b'*'):
             raise Exception("listening socket did not respond with ack byte: response={}".format(read_bytes))
 
         return sock, lambda: socket_closer(sock)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19510.55005.patch
Type: text/x-patch
Size: 1619 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160426/a31e43fd/attachment.bin>