On Fri, 19 Sep 2014 20:18:51 +0200, <[email protected]> wrote:

I think the enum you added goes in "lldb-enumerations.h" not in SBTarget.h.

Ah yes.

Also, I wouldn't use the regex match for the straight match, since that is much less efficient than the ConstString ==, which is just a pointer compare. In the fullness of time, we would retire the version that doesn't take a MatchType, so we don't want that version to be less efficient.

Done.

Also, again, don't forget to add the new definition from SBTarget.h to SBTarget.i or the new API won't be available in Python.


Ah yes, forgot to include that file in the patch. I couldn't file a .i for the enumerations though, I'm not familiar enough with swig to know for sure, but that might get pulled in automatically?


--
Carlo Kok
RemObjects Software

Attachment: find-globals-by-regex3.patch
Description: Binary data

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

Reply via email to