That looks right to me. Jim
> On Aug 18, 2014, at 9:07 PM, Todd Fiala <[email protected]> wrote: > > Hey Jim, > > Does this patch adequately address your comments on the earlier patch? > > Thanks! > > -Todd > > > On Mon, Aug 18, 2014 at 4:14 PM, Todd Fiala <[email protected]> wrote: > Adjust process launch --disable-aslr to take true/false value. > > This change modifies the 'process launch' --disable-aslr option to take a > boolean argument. If the user directly specifies --disable-aslr > {true,false}, that setting will control whether the process is launched with > ASLR disabled accordingly. In the event that the setting is not explicitly > made on the process launch command line, then the value is retrieved from the > target.disable-aslr setting (i.e. settings show target.disable-aslr). > -- > Todd Fiala | Software Engineer | [email protected] | 650-943-3180 > > > > > -- > Todd Fiala | Software Engineer | [email protected] | 650-943-3180 > _______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
