https://github.com/DeinAlptraum closed
https://github.com/llvm/llvm-project/pull/114383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DeinAlptraum wrote:
Closing since this has been split into #114395, #114397, #114399, #114409
https://github.com/llvm/llvm-project/pull/114383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
Endilll wrote:
Well, you fix of the test is not really functional either, but definitely
deserves more attention than the rest of the changes.
https://github.com/llvm/llvm-project/pull/114383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
https://github.com/Endilll requested changes to this pull request.
Thank you for putting effort into this!
You're mixing up non-functional and functional changes here (thank you for
highlighting the mistake you've previously made, it's buried rather deep!), so
I'd like to ask you to split this
@@ -344,7 +343,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)
Dein
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jannick Kremer (DeinAlptraum)
Changes
Clean up imports via isort
Remove unused imports
Remove unused variables
Remove Python <3.6 compatibility measures
---
Patch is 24.66 KiB, truncated to 20.00 KiB below, full version:
https://github.c
https://github.com/DeinAlptraum created
https://github.com/llvm/llvm-project/pull/114383
Clean up imports via isort
Remove unused imports
Remove unused variables
Remove Python <3.6 compatibility measures
>From 25e10c362338265e07ec045b2b8bbe692ae18092 Mon Sep 17 00:00:00 2001
From: Jannick Kreme
