Honsik added inline comments.

================
Comment at: packages/Python/lldbsuite/test/dotest.py:288-294
@@ -288,1 +287,9 @@
+        
+        if sys.platform.startswith('win32'):
+            import ctypes, time
+            while not ctypes.windll.kernel32.IsDebuggerPresent():
+                time.sleep(0.5)
+            ctypes.windll.kernel32.DebugBreak()
+        else:
+            os.kill(os.getpid(), signal.SIGSTOP)
 
----------------
zturner wrote:
> See if you can get [[ https://github.com/Microsoft/PTVS/releases | PTVS ]] to 
> work.  If so, you won't need any of this code and it will be a much better 
> debugging experience anyway.  Assuming PTVS works for you, I'd rather just 
> delete this code.  I'm planning to add info about using PTVS to the website 
> this week.
> 
>  
Ok I removed this part, it is working using PTVS. But maybe in future we will 
still need something similar if we would like to debug lldb testsuite using 
lldb, because I don't know whether lldb will support Python in near future. We 
can always return this part back.


http://reviews.llvm.org/D17492



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to