tberghammer added inline comments.

================
Comment at: include/lldb/Core/RegularExpression.h:34
 #else
-#if __ANDROID_NDK__
+#if ANDROID
 #include <regex>
----------------
In most case you use "#ifdef". It would be nice to uniformalize


================
Comment at: source/Plugins/Language/CPlusPlus/CxxStringTypes.cpp:31
 
-#if __ANDROID_NDK__
+#if ANDROID
 #include <sys/types.h>
----------------
In most case you use "#ifdef". It would be nice to uniformalize


================
Comment at: 
source/Plugins/Process/Utility/RegisterContextMacOSXFrameBackchain.cpp:123
     !defined(_MSC_VER) && !defined(__mips__) && !defined(__powerpc__) &&       
\
-    !defined(__ANDROID_NDK__)
+    !defined(ANDROID)
     case sizeof(long double):
----------------
(unrelated): What does an ANDROID specific define do in a MacOSX specific file?


https://reviews.llvm.org/D27305



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to