2011-04-03 Ivan Maidanski <ivmai@mail.ru>

	* allchblk.c (GC_freehblk): Use GC_log_printf instead of GC_printf
	inside "if (GC_print_stats)" branch.
	* alloc.c (GC_collect_or_expand): Ditto.
	* dyn_load.c (GC_register_dynamic_libraries): Ditto.
	* headers.c (GC_scratch_alloc): Ditto.
	* os_dep.c (GC_get_maps, GC_remap, PROTECT,
	GC_write_fault_handler, GC_dirty_init, GC_mprotect_thread): Ditto.
	* alloc.c (min_bytes_allocd): Use GC_log_printf instead of
	GC_printf for DEBUG_THREADS output.
	* darwin_stop_world.c (GC_stack_range_for, GC_suspend_thread_list,
	GC_stop_world, GC_thread_resume, GC_start_world): Ditto.
	* pthread_start.c (GC_inner_start_routine): Ditto.
	* pthread_stop_world.c (GC_suspend_handler, GC_restart_handler,
	GC_push_all_stacks, GC_suspend_all, GC_stop_world,
	GC_start_world): Ditto.
	* pthread_support.c (GC_mark_thread, GC_get_nprocs,
	GC_start_rtn_prepare_thread, pthread_create): Ditto.
	* alloc.c (GC_adj_bytes_allocd, GC_maybe_gc, GC_stopped_mark,
	GC_finish_collection): Reformat code.
	* pthread_stop_world.c (GC_print_sig_mask): Ditto.
	* pthread_support.c (GC_thr_init): Ditto.
	* checksums.c (GC_update_check_page): Use GC_printf() instead of
	GC_err_printf() for error printing.
	* checksums.c (GC_check_blocks, GC_check_dirty): Use GC_log_printf
	instead of GC_printf for logging purposes.
	* dyn_load.c (sys_errlist, sys_nerr, errno): Move declaration of
	external variable outside from GC_register_dynamic_libraries.
	* dyn_load.c (GC_register_dynamic_libraries): Don't use
	sys_errlist value if errno equals to sys_nerr.
	* dyn_load.c (GC_register_dynamic_libraries): Use GC_log_printf
	instead of GC_printf for DL_VERBOSE output.
	* dyn_load.c (GC_dyld_image_add, GC_dyld_image_remove,
	GC_init_dyld): Use GC_log_printf instead of GC_printf for
	DARWIN_DEBUG output.
	* os_dep.c (catch_exception_raise): Use GC_log_printf
	instead of GC_printf for DEBUG_EXCEPTION_HANDLING output.
	* reclaim.c (GC_print_free_list): Move "n" increment out of
	GC_printf() call.

1 files changed, 16 insertions, 14 deletions

diff --git a/checksums.c b/checksums.c
index e565ccd6..0400697d 100644
--- a/checksums.c
+++ b/checksums.c
@@ -110,7 +110,7 @@ STATIC void GC_update_check_page(struct hblk *h, int index)
     pe -> new_sum = GC_checksum(h);
 #   if !defined(MSWIN32) && !defined(MSWINCE)
         if (pe -> new_sum != 0x80000000 && !GC_page_was_ever_dirty(h)) {
-            GC_printf("GC_page_was_ever_dirty(%p) is wrong\n", h);
+            GC_err_printf("GC_page_was_ever_dirty(%p) is wrong\n", h);
         }
 #   endif
     if (GC_page_was_dirty(h)) {
@@ -165,12 +165,13 @@ STATIC void GC_check_blocks(void)
 
     GC_bytes_in_used_blocks = 0;
     GC_apply_to_all_blocks(GC_add_block, (word)0);
-    GC_printf("GC_bytes_in_used_blocks = %lu, bytes_in_free_blocks = %lu ",
-              (unsigned long)GC_bytes_in_used_blocks,
-              (unsigned long)bytes_in_free_blocks);
-    GC_printf("GC_heapsize = %lu\n", (unsigned long)GC_heapsize);
+    GC_log_printf(
+                "GC_bytes_in_used_blocks = %lu, bytes_in_free_blocks = %lu ",
+                (unsigned long)GC_bytes_in_used_blocks,
+                (unsigned long)bytes_in_free_blocks);
+    GC_log_printf("GC_heapsize = %lu\n", (unsigned long)GC_heapsize);
     if (GC_bytes_in_used_blocks + bytes_in_free_blocks != GC_heapsize) {
-        GC_printf("LOST SOME BLOCKS!!\n");
+        GC_log_printf("LOST SOME BLOCKS!!\n");
     }
 }
 
@@ -202,18 +203,19 @@ void GC_check_dirty(void)
         }
     }
 out:
-    GC_printf("Checked %lu clean and %lu dirty pages\n",
-              (unsigned long) GC_n_clean, (unsigned long) GC_n_dirty);
+    GC_log_printf("Checked %lu clean and %lu dirty pages\n",
+                  (unsigned long)GC_n_clean, (unsigned long)GC_n_dirty);
     if (GC_n_dirty_errors > 0) {
-        GC_printf("Found %d dirty bit errors (%d were faulted)\n",
-                  GC_n_dirty_errors, GC_n_faulted_dirty_errors);
+        GC_log_printf("Found %d dirty bit errors (%d were faulted)\n",
+                      GC_n_dirty_errors, GC_n_faulted_dirty_errors);
     }
     if (GC_n_changed_errors > 0) {
-        GC_printf("Found %lu changed bit errors\n",
-                  (unsigned long)GC_n_changed_errors);
-        GC_printf("These may be benign (provoked by nonpointer changes)\n");
+        GC_log_printf("Found %lu changed bit errors\n",
+                      (unsigned long)GC_n_changed_errors);
+        GC_log_printf(
+                "These may be benign (provoked by nonpointer changes)\n");
 #       ifdef THREADS
-          GC_printf(
+          GC_log_printf(
             "Also expect 1 per thread currently allocating a stubborn obj\n");
 #       endif
     }