Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [lldb][NFC] Remove all `setUp` overrides that only call the parent ↵ | Raphael Isemann | 2019-12-13 | 1 | -4/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | implementation Summary: A lot of our tests copied the setUp code from our TestSampleTest.py: ``` def setUp(self): # Call super's setUp(). TestBase.setUp(self) ``` This code does nothing unless we actually do any setUp work in there, so let's remove all these method definitions. Reviewers: labath, JDevlieghere Reviewed By: labath Subscribers: lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D71454 | ||||
* | [lldb][NFC] Remove 'from __future__ import print_function' from all tests ↵ | Raphael Isemann | 2019-12-13 | 1 | -1/+0 |
| | | | | | | | | | | | | | | | | | | | | | that don't actually call 'print()' Summary: A lot of tests do this trick but the vast majority of them don't even call `print()`. Most of this patch was generated by a script that just looks at all the files and deletes the line if there is no `print (` or `print(` anywhere else in the file. I checked the remaining tests manually and deleted the import if we never call print (but instead do stuff like `expr print(...)` and similar false-positives). I also corrected the additional empty lines after the import in the files that I manually edited. Reviewers: JDevlieghere, labath, jfb Reviewed By: labath Subscribers: dexonsmith, wuzish, nemanjai, kbarton, christof, arphaman, abidh, lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D71452 | ||||
* | [lldb][NFC] Remove unused imports in python tests | Raphael Isemann | 2019-08-02 | 1 | -3/+0 |
| | | | | llvm-svn: 367663 | ||||
* | Fix the swig typemap for "uint32_t *versions, uint32_t num_versions". | Jim Ingham | 2019-03-28 | 2 | -0/+267 |
It was making a list of a certain size but not always filling in that many elements, which would lead to a crash iterating over the list. Differential Revision: https://reviews.llvm.org/D59913 llvm-svn: 357207 |