- Add const keyword / casting to supress more stringient compiler warnings

- Move DynamicImage::Print() from dynamic_images.h to dynamic_images.cc



git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@170 4c0a9323-5329-0410-9bdc-e9ce6186880e

3 files changed, 17 insertions, 15 deletions

diff --git a/src/client/mac/handler/dynamic_images.cc b/src/client/mac/handler/dynamic_images.cc
index e2a9d485..32a57e82 100644
--- a/src/client/mac/handler/dynamic_images.cc
+++ b/src/client/mac/handler/dynamic_images.cc
@@ -104,6 +104,19 @@ void DynamicImage::CalculateMemoryInfo() {
   slide_ = 0;
 }
 
+void DynamicImage::Print() {
+  const char *path = GetFilePath();
+  if (!path) {
+    path = "(unknown)";
+  }
+  printf("%p: %s\n", GetLoadAddress(), path);
+  mach_header *header = GetMachHeader();
+  MachHeader(*header).Print();
+  printf("vmaddr\t\t: %p\n", reinterpret_cast<void*>(GetVMAddr()));
+  printf("vmsize\t\t: %d\n", GetVMSize());
+  printf("slide\t\t: %d\n", GetVMAddrSlide());
+}
+
 #pragma mark -
 
 //==============================================================================
@@ -123,7 +136,7 @@ void DynamicImages::ReadImageInfoForTask() {
   // which lives in "dyld".  This structure contains information about all
   // of the loaded dynamic images.
   struct nlist &list = l[0];
-  list.n_un.n_name = "_dyld_all_image_infos";
+  list.n_un.n_name = const_cast<char *>("_dyld_all_image_infos");
   nlist("/usr/lib/dyld", &list);
   
   if (list.n_value) {
diff --git a/src/client/mac/handler/dynamic_images.h b/src/client/mac/handler/dynamic_images.h
index 2242899c..497f4eb4 100644
--- a/src/client/mac/handler/dynamic_images.h
+++ b/src/client/mac/handler/dynamic_images.h
@@ -147,19 +147,8 @@ class DynamicImage {
   }
 
   // Debugging
-  void Print() {
-    char *path = GetFilePath();
-    if (!path) {
-      path = "(unknown)";
-    }
-    printf("%p: %s\n", GetLoadAddress(), path);
-    mach_header *header = GetMachHeader();
-    MachHeader(*header).Print();
-    printf("vmaddr\t\t: %p\n", reinterpret_cast<void*>(GetVMAddr()));
-    printf("vmsize\t\t: %d\n", GetVMSize());
-    printf("slide\t\t: %d\n", GetVMAddrSlide());
-  }
-
+  void Print();
+ 
  private:
   friend class DynamicImages;
 
diff --git a/src/client/mac/handler/minidump_generator.cc b/src/client/mac/handler/minidump_generator.cc
index e9f1974b..11a770ee 100644
--- a/src/client/mac/handler/minidump_generator.cc
+++ b/src/client/mac/handler/minidump_generator.cc
@@ -594,7 +594,7 @@ bool MinidumpGenerator::WriteCVRecord(MDRawModule *module, int cpu_type,
   TypedMDRVA<MDCVInfoPDB70> cv(&writer_);
 
   // Only return the last path component of the full module path
-  char *module_name = strrchr(module_path, '/');
+  const char *module_name = strrchr(module_path, '/');
 
   // Increment past the slash
   if (module_name)