[all-commits] [llvm/llvm-project] 4a8cc2: Fix TestProcessAPI.py to only allocate sys.maxsize...

Jason Molenda via All-commits all-commits at lists.llvm.org
Tue Feb 7 16:05:38 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4a8cc285e9f3252ca31941dccfddf8fd10c9911b
      https://github.com/llvm/llvm-project/commit/4a8cc285e9f3252ca31941dccfddf8fd10c9911b
  Author: Jason Molenda <jason at molenda.com>
  Date:   2023-02-07 (Tue, 07 Feb 2023)

  Changed paths:
    M lldb/test/API/python_api/process/TestProcessAPI.py

  Log Message:
  -----------
  Fix TestProcessAPI.py to only allocate sys.maxsize buffer

I hardcoded nearly a UINT64_MAX number in this test case,
and python is not able to convert it to a long on some
platforms.  Use sys.maxsize instead; this also would have
failed if the testsuite was run on a 32-bit system.


  Commit: b1d8f40484dfcb28b19c83aa33a674308b17e5dc
      https://github.com/llvm/llvm-project/commit/b1d8f40484dfcb28b19c83aa33a674308b17e5dc
  Author: Jason Molenda <jason at molenda.com>
  Date:   2023-02-07 (Tue, 07 Feb 2023)

  Changed paths:
    M lldb/test/API/python_api/process/TestProcessAPI.py

  Log Message:
  -----------
  Only run the weird new try-to-read-too-much test on Darwin

I'm still getting linux CI bot failures for this test.  It's not
critical, and it depends on a failure mode that is true on Darwin
but I was always gambling that it might fail in the same way on
other systems.


Compare: https://github.com/llvm/llvm-project/compare/8cbf041ecb1c...b1d8f40484df


More information about the All-commits mailing list