[Lldb-commits] [PATCH] D58177: Fix lldb-server test suite for python3

serge via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Feb 13 07:11:51 PST 2019


serge-sans-paille added inline comments.


================
Comment at: packages/Python/lldbsuite/test/tools/lldb-server/lldbgdbserverutils.py:415
             # Add zero-fill to the right/end (MSB side) of the value.
-            retval += "00" * (byte_size - len(retval) / 2)
+            retval += "00" * (byte_size - len(retval) // 2)
         return retval
----------------
Maybe `from __future__ import division` to make sure the `/` operator behaves consistently across versions? That way it may uncover missed `/` to `//` conversion?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58177/new/

https://reviews.llvm.org/D58177





More information about the lldb-commits mailing list