[Lldb-commits] [PATCH] Fix TestStubSetSID after unification in to a single lldb-server binary.
Vince Harron
vharron at google.com
Fri Feb 20 15:28:55 PST 2015
REPOSITORY
rL LLVM
================
Comment at: lldb/trunk/test/tools/lldb-gdbserver/commandline/TestStubSetSID.py:42
@@ -36,3 +41,3 @@
def sid_is_different_with_setsid(self):
- stub_sid = self.get_stub_sid(" --setsid")
+ stub_sid = self.get_stub_sid(" %s --setsid" % ' '.join(get_common_stub_args()))
self.assertNotEquals(stub_sid, os.getsid(0))
----------------
```
since there's only 0-1 elements in the array, why bother with the join? Why not just
# returns "g" to make lldb-server run in gdbserver mode
def get_debugserver_mode():
return "" if 'darwin' in sys.platform else "g"
stub_sid = self.get_stub_sid(' %{mode} --setsid'.format(mode=get_debugserver_mode()))
```
http://reviews.llvm.org/D7801
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the lldb-commits
mailing list