Skip to content

Conversation

@Endilll
Copy link
Contributor

@Endilll Endilll commented May 16, 2025

The list is not that long.

@Endilll Endilll requested a review from DeinAlptraum as a code owner May 16, 2025 05:13
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:as-a-library libclang and C++ API labels May 16, 2025
@llvmbot
Copy link
Member

llvmbot commented May 16, 2025

@llvm/pr-subscribers-clang

Author: Vlad Serebrennikov (Endilll)

Changes

The list is not that long.


Full diff: https://github.com/llvm/llvm-project/pull/140191.diff

1 Files Affected:

  • (modified) clang/bindings/python/clang/cindex.py (+18-1)
diff --git a/clang/bindings/python/clang/cindex.py b/clang/bindings/python/clang/cindex.py
index 4ff7f318416b7..1bbe006fcb1b9 100644
--- a/clang/bindings/python/clang/cindex.py
+++ b/clang/bindings/python/clang/cindex.py
@@ -62,7 +62,24 @@
 #
 # o implement additional SourceLocation, SourceRange, and File methods.
 
-from ctypes import *
+from ctypes import (
+    Array,
+    CDLL,
+    CFUNCTYPE,
+    POINTER,
+    Structure,
+    byref,
+    c_char_p,
+    c_int,
+    c_longlong,
+    c_uint,
+    c_ulong,
+    c_ulonglong,
+    c_void_p,
+    cast,
+    cdll,
+    py_object,
+)
 
 import os
 import sys

Copy link
Contributor

@DeinAlptraum DeinAlptraum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea! LGTM

@Endilll Endilll merged commit 17853c3 into llvm:main May 16, 2025
16 checks passed
@Endilll Endilll deleted the replace-ctypes-wildcard-import branch May 16, 2025 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:as-a-library libclang and C++ API clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants