Skip to content

Conversation

@DeinAlptraum
Copy link
Contributor

This mistake was introduced in #109846

@DeinAlptraum DeinAlptraum added the clang:as-a-library libclang and C++ API label Oct 31, 2024
@DeinAlptraum DeinAlptraum requested a review from Endilll October 31, 2024 12:09
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Oct 31, 2024
@llvmbot
Copy link
Member

llvmbot commented Oct 31, 2024

@llvm/pr-subscribers-clang

Author: Jannick Kremer (DeinAlptraum)

Changes

This mistake was introduced in #109846


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

1 Files Affected:

  • (modified) clang/bindings/python/tests/cindex/test_cursor.py (+1-1)
diff --git a/clang/bindings/python/tests/cindex/test_cursor.py b/clang/bindings/python/tests/cindex/test_cursor.py
index 77d8ca415708f8..f118613e3209d2 100644
--- a/clang/bindings/python/tests/cindex/test_cursor.py
+++ b/clang/bindings/python/tests/cindex/test_cursor.py
@@ -344,7 +344,7 @@ class Bar {
         )
 
         self.assertEqual(len(copy_assignment_operators_cursors), 10)
-        self.assertTrue(len(non_copy_assignment_operators_cursors), 9)
+        self.assertEqual(len(non_copy_assignment_operators_cursors), 7)
 
         self.assertTrue(
             all(

@DeinAlptraum DeinAlptraum merged commit d0ffb53 into llvm:main Oct 31, 2024
11 of 13 checks passed
@DeinAlptraum DeinAlptraum deleted the fix-assert branch October 31, 2024 14:20
smallp-o-p pushed a commit to smallp-o-p/llvm-project that referenced this pull request Nov 3, 2024
NoumanAmir657 pushed a commit to NoumanAmir657/llvm-project that referenced this pull request Nov 4, 2024
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