In http://reviews.llvm.org/D6965#110399, @abidh wrote:

> 1. Why are changes in SBTarget when you are not using them.


Oops. It's the old version. I'll update patch right now.

> 2. I think the test cases for arguments are currently skipped 
> (@unittest2.skip("requires 'quotes' and 'CLI support' patches")). Currently, 
> we can test for argc. So I think at least that portion should be enabled as 
> that gives us some coverage. I will have to see why argv is not being 
> evaluated.


Ok. I think that I can fix this test using -interpreter-exec command.


http://reviews.llvm.org/D6965

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to