My suggestion was to have the SBTarget vend the SBLaunchInfo it WOULD use if 
you were to call launch without providing an SBLaunchInfo of your own.  
Unfortunately, the underlying target doesn't actually use a ProcessLaunch info 
to store these settings, it stores the individual bits.  That's purely 
historical: the ProcessLaunchInfo was a later invention, and was layered onto 
of the existing code without re-working it.   We should probably change that at 
some point.

So if you want extra credit it would be great to re-plumb this stuff.  But I 
can understand if that's way more than you thought you were biting off for this 
task.  I'm still unhappy about adding yet another way to set run arguments, and 
once it's in the SB API's we're stuck with it for a while.

But given that it's our fault we are in this situation, it doesn't seem fair to 
push this too hard.

Jim


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