From: Ivan Maidanski <ivmai@mail.ru>
Date: Wed, 12 Dec 2012 14:52:14 +0000 (+0400)
Subject: Redirect WRITE to __android_log_write if GC_ANDROID_LOG (Android)
X-Git-Tag: gc7_4_0~118
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=54a91f5254a1c5b2081e753cde47e6fa3a612e23;p=gc

Redirect WRITE to __android_log_write if GC_ANDROID_LOG (Android)
(code refactoring)

* misc.c (GC_stdout, GC_stderr, GC_log): Do not define as static
variable if GC_ANDROID_LOG.
* misc.c (GC_init): Do not allow to set GC_log and GC_stdout/err if
GC_ANDROID_LOG.
* misc.c (GC_ANDROID_LOG_TAG): Move definition (and include
android/log.h) upper to precede GC_write definition for Unix.
* misc.c (GC_stdout, GC_stderr, GC_long): Define specially
for GC_ANDROID_LOG (define to ANDROID_LOG_DEBUG/ERROR logging level).
* misc.c (WRITE): Define specially for GC_ANDROID_LOG (first argument
is used as logging level) redirecting to __android_log_write.
* misc.c (GC_printf, GC_stats_log_printf, GC_verbose_log_printf,
GC_warn_printf, GC_err_puts): Remove __android_log_write call
(unconditionally invoke WRITE instead).
* misc.c (GC_log_printf): Remove Android-specific implementation (use
general instead).
* misc.c (GC_stats_log_printf, GC_verbose_log_printf): Expand
GC_LOG_PRINTF_IMPL macro.
* misc.c (GC_LOG_PRINTF_IMPL): Remove macro.
---

diff --git a/misc.c b/misc.c
index dad7acaa..11e70807 100644
--- a/misc.c
+++ b/misc.c
@@ -773,7 +773,8 @@ STATIC void GC_exit_check(void)
 #define GC_DEFAULT_STDOUT_FD 1
 #define GC_DEFAULT_STDERR_FD 2
 
-#if !defined(OS2) && !defined(MACOS) && !defined(MSWIN32) && !defined(MSWINCE)
+#if !defined(OS2) && !defined(MACOS) && !defined(GC_ANDROID_LOG) \
+    && !defined(MSWIN32) && !defined(MSWINCE)
   STATIC int GC_stdout = GC_DEFAULT_STDOUT_FD;
   STATIC int GC_stderr = GC_DEFAULT_STDERR_FD;
   STATIC int GC_log = GC_DEFAULT_STDERR_FD;
@@ -892,7 +893,8 @@ GC_API void GC_CALL GC_init(void)
           GC_print_stats = 1;
         }
 #     endif
-#     if defined(UNIX_LIKE) || defined(CYGWIN32) || defined(SYMBIAN)
+#     if (defined(UNIX_LIKE) && !defined(GC_ANDROID_LOG)) \
+         || defined(CYGWIN32) || defined(SYMBIAN)
         {
           char * file_name = GETENV("GC_LOG_FILE");
 #         ifdef GC_LOG_TO_FILE_ALWAYS
@@ -1425,6 +1427,21 @@ GC_API void GC_CALL GC_enable_incremental(void)
 
 # define WRITE(f, buf, len) (GC_set_files(), GC_write(f, buf, len))
 
+#elif defined(GC_ANDROID_LOG)
+
+# include <android/log.h>
+
+# ifndef GC_ANDROID_LOG_TAG
+#   define GC_ANDROID_LOG_TAG "BDWGC"
+# endif
+
+# define GC_stdout ANDROID_LOG_DEBUG
+# define GC_stderr ANDROID_LOG_ERROR
+# define GC_log GC_stdout
+
+# define WRITE(level, buf, unused_len) \
+                __android_log_write(level, GC_ANDROID_LOG_TAG, buf)
+
 #else
 # if !defined(AMIGA) && !defined(__CC_ARM)
 #   include <unistd.h>
@@ -1465,15 +1482,7 @@ GC_API void GC_CALL GC_enable_incremental(void)
   }
 
 # define WRITE(f, buf, len) GC_write(f, buf, len)
-#endif /* !MSWIN32 && !OS2 && !MACOS */
-
-#ifdef GC_ANDROID_LOG
-# include <android/log.h>
-
-# ifndef GC_ANDROID_LOG_TAG
-#   define GC_ANDROID_LOG_TAG "BDWGC"
-# endif
-#endif
+#endif /* !MSWIN32 && !OS2 && !MACOS && !GC_ANDROID_LOG */
 
 #define BUFSZ 1024
 
@@ -1510,12 +1519,8 @@ void GC_printf(const char *format, ...)
 
     if (!GC_quiet) {
       GC_PRINTF_FILLBUF(buf, format);
-#   ifndef GC_ANDROID_LOG
       if (WRITE(GC_stdout, buf, strlen(buf)) < 0)
         ABORT("write to stdout failed");
-#   else
-      __android_log_write(ANDROID_LOG_DEBUG, GC_ANDROID_LOG_TAG, buf);
-#   endif
     }
 }
 
@@ -1527,41 +1532,35 @@ void GC_err_printf(const char *format, ...)
     GC_err_puts(buf);
 }
 
-#ifndef GC_ANDROID_LOG
-
-  void GC_log_printf(const char *format, ...)
-  {
+void GC_log_printf(const char *format, ...)
+{
     char buf[BUFSZ + 1];
 
     GC_PRINTF_FILLBUF(buf, format);
     if (WRITE(GC_log, buf, strlen(buf)) < 0)
       ABORT("write to GC log failed");
-  }
+}
+
+#ifndef GC_ANDROID_LOG
 
 # define GC_warn_printf GC_err_printf
 
 #else
 
-# define GC_LOG_PRINTF_IMPL(loglevel, format) \
-        { \
-          char buf[BUFSZ + 1]; \
-          GC_PRINTF_FILLBUF(buf, format); \
-          __android_log_write(loglevel, GC_ANDROID_LOG_TAG, buf); \
-        }
-
-  void GC_log_printf(const char *format, ...)
-  {
-    GC_LOG_PRINTF_IMPL(ANDROID_LOG_DEBUG, format);
-  }
-
   GC_INNER void GC_stats_log_printf(const char *format, ...)
   {
-    GC_LOG_PRINTF_IMPL(ANDROID_LOG_INFO, format);
+    char buf[BUFSZ + 1];
+
+    GC_PRINTF_FILLBUF(buf, format);
+    (void)WRITE(ANDROID_LOG_INFO, buf, 0 /* unused */);
   }
 
   GC_INNER void GC_verbose_log_printf(const char *format, ...)
   {
-    GC_LOG_PRINTF_IMPL(ANDROID_LOG_VERBOSE, format);
+    char buf[BUFSZ + 1];
+
+    GC_PRINTF_FILLBUF(buf, format);
+    (void)WRITE(ANDROID_LOG_VERBOSE, buf, 0); /* ignore write errors */
   }
 
   STATIC void GC_warn_printf(const char *format, ...)
@@ -1569,18 +1568,14 @@ void GC_err_printf(const char *format, ...)
     char buf[BUFSZ + 1];
 
     GC_PRINTF_FILLBUF(buf, format);
-    __android_log_write(ANDROID_LOG_WARN, GC_ANDROID_LOG_TAG, buf);
+    (void)WRITE(ANDROID_LOG_WARN, buf, 0);
   }
 
 #endif /* GC_ANDROID_LOG */
 
 void GC_err_puts(const char *s)
 {
-# ifdef GC_ANDROID_LOG
-      __android_log_write(ANDROID_LOG_ERROR, GC_ANDROID_LOG_TAG, s);
-# else
     (void)WRITE(GC_stderr, s, strlen(s)); /* ignore errors */
-# endif
 }
 
 STATIC void GC_CALLBACK GC_default_warn_proc(char *msg, GC_word arg)