[Lldb-commits] [PATCH] D43024: [test] Enable test category for inline tests.

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Feb 7 08:16:08 PST 2018

labath added inline comments.

Comment at: lldb/packages/Python/lldbsuite/test/decorators.py:307-308
                 "@add_test_categories can only be used to decorate a test method")
         if hasattr(func, "categories"):
+        # For instance methods, the attribute must be set on the actual function.
These two lines look like they need to be fixed as well.

Comment at: lldb/packages/Python/lldbsuite/test/decorators.py:526
                 return None
         header = os.path.join(
davide wrote:
> We should consider clang-formatting this file altogether.
The current (weird) formatting is the result of running `autopep8` as a part of the huge refactor some time ago...



More information about the lldb-commits mailing list