Thanks for the bug report and patch. A couple of things. First, the commentary is big enough so that we'd need a copyright assignment for this -- if you and your employer are OK with that please let me know and I'll start the ball rolling on the paperwork. If this is an issue we can rederive just the code change (itself trivial) from the bug report so it's no big deal.

Second, I'd like to avoid having stdint depend on sys_types. Does the attached smaller change work for you? If so, I can push it along with commentary that I can write up.
diff --git a/lib/stdint.in.h b/lib/stdint.in.h
index 91fad37..90545d5 100644
--- a/lib/stdint.in.h
+++ b/lib/stdint.in.h
@@ -39,7 +39,7 @@
    Ideally we should test __BIONIC__ here, but it is only defined after
    <sys/cdefs.h> has been included; hence test __ANDROID__ instead.  */
 #if defined __ANDROID__ \
-    && defined _SYS_TYPES_H_ && !defined __need_size_t
+    && defined _SYS_TYPES_H_ && !defined _SSIZE_T_DEFINED_
 # @INCLUDE_NEXT@ @NEXT_STDINT_H@
 #else
 

Reply via email to