labath accepted this revision. labath added a comment. This revision is now accepted and ready to land.
Looks good, after applying one fix. ================ Comment at: packages/Python/lldbsuite/test/decorators.py:148 @@ +147,3 @@ + skip_for_triple = _match_decorator_property(triple, lldb.DBG.GetSelectedPlatform().GetTriple()) + skip_for_remote = _match_decorator_property(remote, remote == (lldb.remote_platform is not None)) + ---------------- The `actual` value should be simply `lldb.remote_platform is not None`. When remote_platform is None, we are local, when it is not None, we are remote. Btw, since this flag can only have three values (None, True, False), the _match_decorator property dance is not really necessary. Although, I guess it won't hurt either... http://reviews.llvm.org/D16938 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
