[Lldb-commits] [PATCH] D16872: Move some of the common decorators to decorators.py

[Tamas Berghammer via lldb-commits]

tberghammer accepted this revision.
tberghammer added a comment.

Looks reasonable


================
Comment at: packages/Python/lldbsuite/test/lldbtest.py:672-682
@@ -768,1 +671,13 @@
+
+def skipUnlessListedRemote(remote_list=None):
+    def is_remote_unlisted(self):
+        if remote_list and lldb.remote_platform:
+            triple = self.dbg.GetSelectedPlatform().GetTriple()
+            for r in remote_list:
+                if r in triple:
+                    return None
+            return "skipping because remote is not listed"
+        else:
+            return None
+    return decorators.skipTestIfFn(is_remote_unlisted)
 
----------------
I think Pavel intentionally removed this function in a recent CL. Please don't add it back


http://reviews.llvm.org/D16872