[PATCH] [DbgHelp]: get rid of module_name in all traces (using Unicode flavor)

Eric Pouech eric.pouech at wanadoo.fr
Tue Mar 13 11:26:33 CDT 2007




A+
---

 dlls/dbghelp/dwarf.c      |    6 ++++--
 dlls/dbghelp/elf_module.c |   20 ++++++++++++--------
 dlls/dbghelp/module.c     |    2 +-
 dlls/dbghelp/msc.c        |    2 +-
 dlls/dbghelp/pe_module.c  |    2 +-
 dlls/dbghelp/symbol.c     |   16 ++++++++--------
 dlls/dbghelp/type.c       |    3 ++-
 7 files changed, 29 insertions(+), 22 deletions(-)

diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c
index e4944e3..3f2f5c7 100644
--- a/dlls/dbghelp/dwarf.c
+++ b/dlls/dbghelp/dwarf.c
@@ -330,7 +330,8 @@ static const char* dwarf2_debug_traverse
 
 static const char* dwarf2_debug_ctx(const dwarf2_parse_context_t* ctx)
 {
-    return wine_dbg_sprintf("ctx(%p,%s)", ctx, ctx->module->module_name);
+    return wine_dbg_sprintf("ctx(%p,%s)",
+                            ctx, debugstr_w(ctx->module->module.ModuleName));
 }
 
 static const char* dwarf2_debug_di(const dwarf2_debug_info_t* di)
@@ -1749,7 +1750,8 @@ static void dwarf2_set_line_number(struc
 
     if (!file || !(psrc = vector_at(v, file - 1))) return;
 
-    TRACE("%s %lx %s %u\n", module->module_name, address, source_get(module, *psrc), line);
+    TRACE("%s %lx %s %u\n",
+          debugstr_w(module->module.ModuleName), address, source_get(module, *psrc), line);
     if (!(symt = symt_find_nearest(module, address)) ||
         symt->symt.tag != SymTagFunction) return;
     func = (struct symt_function*)symt;
diff --git a/dlls/dbghelp/elf_module.c b/dlls/dbghelp/elf_module.c
index 4b67ee1..d509db6 100644
--- a/dlls/dbghelp/elf_module.c
+++ b/dlls/dbghelp/elf_module.c
@@ -529,7 +529,7 @@ static const Elf32_Sym* elf_lookup_symta
     if (!result && !(result = weak_result))
     {
         FIXME("Couldn't find symbol %s!%s in symtab\n",
-              module->module_name, name);
+              debugstr_w(module->module.ModuleName), name);
         return NULL;
     }
     return result->symp;
@@ -567,17 +567,19 @@ static void elf_finish_stabs_info(struct
                 if (((struct symt_function*)sym)->address != module->elf_info->elf_addr &&
                     ((struct symt_function*)sym)->address != module->elf_info->elf_addr + symp->st_value)
                     FIXME("Changing address for %p/%s!%s from %08lx to %08lx\n",
-                          sym, module->module_name, sym->hash_elt.name,
+                          sym, debugstr_w(module->module.ModuleName), sym->hash_elt.name,
                           ((struct symt_function*)sym)->address, module->elf_info->elf_addr + symp->st_value);
                 if (((struct symt_function*)sym)->size && ((struct symt_function*)sym)->size != symp->st_size)
                     FIXME("Changing size for %p/%s!%s from %08lx to %08x\n",
-                          sym, module->module_name, sym->hash_elt.name,
+                          sym, debugstr_w(module->module.ModuleName), sym->hash_elt.name,
                           ((struct symt_function*)sym)->size, symp->st_size);
 
                 ((struct symt_function*)sym)->address = module->elf_info->elf_addr +
                                                         symp->st_value;
                 ((struct symt_function*)sym)->size    = symp->st_size;
-            } else FIXME("Couldn't find %s!%s\n", module->module_name, sym->hash_elt.name);
+            } else
+                FIXME("Couldn't find %s!%s\n",
+                      debugstr_w(module->module.ModuleName), sym->hash_elt.name);
             break;
         case SymTagData:
             switch (((struct symt_data*)sym)->kind)
@@ -593,13 +595,15 @@ static void elf_finish_stabs_info(struct
                 if (((struct symt_data*)sym)->u.var.offset != module->elf_info->elf_addr &&
                     ((struct symt_data*)sym)->u.var.offset != module->elf_info->elf_addr + symp->st_value)
                     FIXME("Changing address for %p/%s!%s from %08lx to %08lx\n",
-                          sym, module->module_name, sym->hash_elt.name,
+                          sym, debugstr_w(module->module.ModuleName), sym->hash_elt.name,
                           ((struct symt_function*)sym)->address, module->elf_info->elf_addr + symp->st_value);
                     ((struct symt_data*)sym)->u.var.offset = module->elf_info->elf_addr +
                                                           symp->st_value;
                     ((struct symt_data*)sym)->kind = (ELF32_ST_BIND(symp->st_info) == STB_LOCAL) ?
                         DataIsFileStatic : DataIsGlobal;
-                } else FIXME("Couldn't find %s!%s\n", module->module_name, sym->hash_elt.name);
+                } else
+                    FIXME("Couldn't find %s!%s\n",
+                          debugstr_w(module->module.ModuleName), sym->hash_elt.name);
                 break;
             default:;
             }
@@ -694,7 +698,7 @@ static int elf_new_wine_thunks(struct mo
                 if ((xsize || ste->symp->st_size) &&
                     (kind == (ELF32_ST_BIND(ste->symp->st_info) == STB_LOCAL) ? DataIsFileStatic : DataIsGlobal))
                     FIXME("Duplicate in %s: %s<%08x-%08x> %s<%s-%s>\n",
-                          module->module_name,
+                          debugstr_w(module->module.ModuleName),
                           ste->ht_elt.name, addr, ste->symp->st_size,
                           symt->hash_elt.name,
                           wine_dbgstr_longlong(xaddr), wine_dbgstr_longlong(xsize));
@@ -1055,7 +1059,7 @@ static BOOL elf_load_debug_info_from_map
             const BYTE* dw2_debug_line;
             const BYTE* dw2_debug_loclist;
 
-            TRACE("Loading Dwarf2 information for %s\n", module->module_name);
+            TRACE("Loading Dwarf2 information for %s\n", debugstr_w(module->module.ModuleName));
 
 	    elf_find_section(fmap, ".debug_str", SHT_NULL, &debug_str_sect);
 	    elf_find_section(fmap, ".debug_abbrev", SHT_NULL, &debug_abbrev_sect);
diff --git a/dlls/dbghelp/module.c b/dlls/dbghelp/module.c
index c906d1f..9a5fa24 100644
--- a/dlls/dbghelp/module.c
+++ b/dlls/dbghelp/module.c
@@ -545,7 +545,7 @@ BOOL module_remove(struct process* pcs, 
 {
     struct module**     p;
 
-    TRACE("%s (%p)\n", module->module_name, module);
+    TRACE("%s (%p)\n", debugstr_w(module->module.ModuleName), module);
     hash_table_destroy(&module->ht_symbols);
     hash_table_destroy(&module->ht_types);
     HeapFree(GetProcessHeap(), 0, (char*)module->sources);
diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c
index 00ba7c2..dcd24e9 100644
--- a/dlls/dbghelp/msc.c
+++ b/dlls/dbghelp/msc.c
@@ -2444,7 +2444,7 @@ static BOOL codeview_process_info(const 
     }
     default:
         ERR("Unknown CODEVIEW signature %.4s in module %s\n",
-            (const char*)signature, msc_dbg->module->module_name);
+            (const char*)signature, debugstr_w(msc_dbg->module->module.ModuleName));
         break;
     }
     if (ret)
diff --git a/dlls/dbghelp/pe_module.c b/dlls/dbghelp/pe_module.c
index f8346d1..8f75b80 100644
--- a/dlls/dbghelp/pe_module.c
+++ b/dlls/dbghelp/pe_module.c
@@ -178,7 +178,7 @@ static BOOL pe_load_msc_debug_info(const
             misc->DataType != IMAGE_DEBUG_MISC_EXENAME)
         {
             WINE_ERR("-Debug info stripped, but no .DBG file in module %s\n",
-                     module->module_name);
+                     debugstr_w(module->module.ModuleName));
         }
         else
         {
diff --git a/dlls/dbghelp/symbol.c b/dlls/dbghelp/symbol.c
index 06240d4..90f9d8f 100644
--- a/dlls/dbghelp/symbol.c
+++ b/dlls/dbghelp/symbol.c
@@ -131,7 +131,7 @@ struct symt_compiland* symt_new_compilan
     struct symt_compiland*    sym;
 
     TRACE_(dbghelp_symt)("Adding compiland symbol %s:%s\n",
-                         module->module_name, source_get(module, src_idx));
+                         debugstr_w(module->module.ModuleName), source_get(module, src_idx));
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
     {
         sym->symt.tag = SymTagCompiland;
@@ -152,7 +152,7 @@ struct symt_public* symt_new_public(stru
     struct symt**       p;
 
     TRACE_(dbghelp_symt)("Adding public symbol %s:%s @%lx\n",
-                         module->module_name, name, address);
+                         debugstr_w(module->module.ModuleName), name, address);
     if ((dbghelp_options & SYMOPT_AUTO_PUBLICS) &&
         symt_find_nearest(module, address) != NULL)
         return NULL;
@@ -187,7 +187,7 @@ struct symt_data* symt_new_global_variab
     DWORD64             tsz;
 
     TRACE_(dbghelp_symt)("Adding global symbol %s:%s @%lx %p\n",
-                         module->module_name, name, addr, type);
+                         debugstr_w(module->module.ModuleName), name, addr, type);
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
     {
         sym->symt.tag      = SymTagData;
@@ -202,7 +202,7 @@ struct symt_data* symt_new_global_variab
         {
             if (tsz != size)
                 FIXME("Size mismatch for %s.%s between type (%s) and src (%lu)\n",
-                      module->module_name, name,
+                      debugstr_w(module->module.ModuleName), name,
                       wine_dbgstr_longlong(tsz), size);
         }
         if (compiland)
@@ -224,7 +224,7 @@ struct symt_function* symt_new_function(
     struct symt**               p;
 
     TRACE_(dbghelp_symt)("Adding global function %s:%s @%lx-%lx\n",
-                         module->module_name, name, addr, addr + size - 1);
+                         debugstr_w(module->module.ModuleName), name, addr, addr + size - 1);
 
     assert(!sig_type || sig_type->tag == SymTagFunctionType);
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
@@ -310,7 +310,7 @@ struct symt_data* symt_add_func_local(st
     struct symt**       p;
 
     TRACE_(dbghelp_symt)("Adding local symbol (%s:%s): %s %p\n",
-                         module->module_name, func->hash_elt.name,
+                         debugstr_w(module->module.ModuleName), func->hash_elt.name,
                          name, type);
 
     assert(func);
@@ -425,7 +425,7 @@ struct symt_thunk* symt_new_thunk(struct
     struct symt_thunk*  sym;
 
     TRACE_(dbghelp_symt)("Adding global thunk %s:%s @%lx-%lx\n",
-                         module->module_name, name, addr, addr + size - 1);
+                         debugstr_w(module->module.ModuleName), name, addr, addr + size - 1);
 
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
     {
@@ -455,7 +455,7 @@ struct symt_data* symt_new_constant(stru
     struct symt_data*  sym;
 
     TRACE_(dbghelp_symt)("Adding constant value %s:%s\n",
-                         module->module_name, name);
+                         debugstr_w(module->module.ModuleName), name);
 
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
     {
diff --git a/dlls/dbghelp/type.c b/dlls/dbghelp/type.c
index 4a340ad..7429b04 100644
--- a/dlls/dbghelp/type.c
+++ b/dlls/dbghelp/type.c
@@ -164,7 +164,8 @@ struct symt_udt* symt_new_udt(struct mod
 {
     struct symt_udt*            sym;
 
-    TRACE_(dbghelp_symt)("Adding udt %s:%s\n", module->module_name, typename);
+    TRACE_(dbghelp_symt)("Adding udt %s:%s\n",
+                         debugstr_w(module->module.ModuleName), typename);
     if ((sym = pool_alloc(&module->pool, sizeof(*sym))))
     {
         sym->symt.tag = SymTagUDT;



More information about the wine-patches mailing list