[all-commits] [llvm/llvm-project] c49770: [NFC] Prefer subprocess.DEVNULL over os.devnull (#...

Nicolas van Kempen via All-commits all-commits at lists.llvm.org
Fri Aug 30 11:27:10 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c49770c60f26e449379447109f7d915bd8de0384
      https://github.com/llvm/llvm-project/commit/c49770c60f26e449379447109f7d915bd8de0384
  Author: Nicolas van Kempen <nvankemp at gmail.com>
  Date:   2024-08-30 (Fri, 30 Aug 2024)

  Changed paths:
    M clang-tools-extra/clang-tidy/tool/run-clang-tidy.py
    M clang/docs/tools/generate_formatted_state.py
    M clang/tools/scan-view/share/startfile.py
    M clang/utils/creduce-clang-crash.py
    M lldb/bindings/interface/SBErrorDocstrings.i
    M lldb/packages/Python/lldbsuite/test/decorators.py
    M lldb/packages/Python/lldbsuite/test/lldbtest.py
    M llvm/utils/UpdateTestChecks/common.py
    M llvm/utils/git/pre-push.py
    M llvm/utils/gn/gn.py

  Log Message:
  -----------
  [NFC] Prefer subprocess.DEVNULL over os.devnull (#106500)

There is no need to support Python 2.7 anymore, Python 3.3+ has
`subprocess.DEVNULL`. This is good practice and also prevents file
handles from
staying open unnecessarily.

Also remove a couple unused or unneeded `__future__` imports.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list