[Lldb-commits] [PATCH] D67073: [dotest] Add @skipIfCursesSupportMissing and annotate the new gui test

Jan Kratochvil via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 2 05:39:55 PDT 2019


jankratochvil accepted this revision.
jankratochvil added inline comments.


================
Comment at: packages/Python/lldbsuite/test/decorators.py:786
 
-def skipIfXmlSupportMissing(func):
+def _get_bool_config_decorator(key):
     config = lldb.SBDebugger.GetBuildConfiguration()
----------------
I find a bit confusing the function returns opposite of `key` existence but its name does not indicate that.  The `skipIf` could stay in its name.



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67073/new/

https://reviews.llvm.org/D67073





More information about the lldb-commits mailing list