tfiala added inline comments. ================ Comment at: packages/Python/lldbsuite/test/dosep.py:1145 @@ -1144,1 +1144,3 @@ + else: + target = None ---------------- labath wrote: > I think this should be `target = platform_name`, so that you can still match > the ios-simulator platform below (the point of the regex was to remove the > "remote-" prefix, I believe), but I guess it doesn't really matter, as we are > getting rid of the whole function anyway... Yep, I agree on both counts: 1. I'll change, 2. It won't matter if/when Ed has a look at this since I don't think we'll need to call out expected timeouts this way.
I'll adjust and submit the adjustment after I test it. Thanks! http://reviews.llvm.org/D16423 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits