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"):
             cat.extend(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...


Repository:
  rL LLVM

https://reviews.llvm.org/D43024



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to