[Lldb-commits] [PATCH] Make llgs build on Android. No functionality changes.

Tong Shen endlessroad at google.com
Thu Sep 25 16:30:36 PDT 2014


Thanks for the comment, Jim!

IOHandlerEditLine:
Yes there's no need to use IOHandlerEditLine in that place; it just
parses commands and execute them.
I will come up with a new subclass of IOHandler and replace
IOHandlerEditLine there.
Can I do that in a follow-up patch, or do you prefer I do it in this patch?

Python goo:
Sure, I will use LLDB_DISABLE_PYTHON.

Other uses:
Are you strongly against any other use of __ANDROID__?


On Thu, Sep 25, 2014 at 4:23 PM,  <jingham at apple.com> wrote:
> I don't have time to look at this right now, but that's an awful lot of #ifndef __ANDROID__ in places I wouldn't really expect these defines.  Particularly the ones that deal with the IOHandlerEditLine.  Either you're building stuff for llgs that you don't really need to (why does it need the command interpreter) or we really need a non-interactive configuration for the command interpreter that substitutes the EditLine I/O handler with one that is not interactive...  Anyway, that part does seem ugly to me.
>
> Also one of the uses of __ANDROID__ is to not include some python goo, but there's already a "build without python" define for that purpose.
>
> Jim
>
>
>> On Sep 25, 2014, at 4:15 PM, Todd Fiala <tfiala at google.com> wrote:
>>
>> LGTM.
>>
>> We might later want to look at how/where we're constructing IOHandlerEditLine.  It seems like it might first show up at too low a level of the stack based on your need to comment it out (unneeded in llgs and not wanting to bring in the libedit dependency).
>>
>> I'll ping Greg to see if he wants somebody to look over before we approve and move this in.
>>
>> http://reviews.llvm.org/D5495
>>
>>
>>
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>



-- 
Best Regards, Tong Shen




More information about the lldb-commits mailing list