Commit 8430ebc6 authored by Ken Thomases's avatar Ken Thomases Committed by Alexandre Julliard

dbghelp: Eliminate an unnecessary parameter from macho_load_debug_info().

From outside of macho_module.c, it's only called with NULL. When it's called with a non-NULL parameter, it's just a thin wrapper around macho_load_debug_info_from_map(), so the code can just call that directly.
parent 604ce29c
......@@ -572,8 +572,7 @@ extern int elf_is_in_thunk_area(unsigned long addr, const struct elf_th
#define MACHO_NO_MAP ((const void*)-1)
extern BOOL macho_enum_modules(HANDLE hProc, enum_modules_cb, void*) DECLSPEC_HIDDEN;
extern BOOL macho_fetch_file_info(const WCHAR* name, DWORD_PTR* base, DWORD* size, DWORD* checksum) DECLSPEC_HIDDEN;
struct macho_file_map;
extern BOOL macho_load_debug_info(struct module* module, struct macho_file_map* fmap) DECLSPEC_HIDDEN;
extern BOOL macho_load_debug_info(struct module* module) DECLSPEC_HIDDEN;
extern struct module*
macho_load_module(struct process* pcs, const WCHAR* name, unsigned long) DECLSPEC_HIDDEN;
extern BOOL macho_read_wine_loader_dbg_info(struct process* pcs) DECLSPEC_HIDDEN;
......
......@@ -901,12 +901,12 @@ static BOOL macho_load_debug_info_from_map(struct module* module,
*
* Loads Mach-O debugging information from the module image file.
*/
BOOL macho_load_debug_info(struct module* module, struct macho_file_map* fmap)
BOOL macho_load_debug_info(struct module* module)
{
BOOL ret = TRUE;
struct macho_file_map my_fmap;
struct macho_file_map fmap;
TRACE("(%p, %p/%d)\n", module, fmap, fmap ? fmap->fd : -1);
TRACE("(%p)\n", module);
if (module->type != DMT_MACHO || !module->format_info[DFI_MACHO]->u.macho_info)
{
......@@ -914,15 +914,11 @@ BOOL macho_load_debug_info(struct module* module, struct macho_file_map* fmap)
return FALSE;
}
if (!fmap)
{
fmap = &my_fmap;
ret = macho_map_file(module->module.LoadedImageName, fmap);
}
ret = macho_map_file(module->module.LoadedImageName, &fmap);
if (ret)
ret = macho_load_debug_info_from_map(module, fmap);
ret = macho_load_debug_info_from_map(module, &fmap);
if (fmap == &my_fmap) macho_unmap_file(fmap);
macho_unmap_file(&fmap);
return ret;
}
......@@ -1048,7 +1044,7 @@ static BOOL macho_load_file(struct process* pcs, const WCHAR* filename,
if (dbghelp_options & SYMOPT_DEFERRED_LOADS)
macho_info->module->module.SymType = SymDeferred;
else if (!macho_load_debug_info(macho_info->module, &fmap))
else if (!macho_load_debug_info_from_map(macho_info->module, &fmap))
ret = FALSE;
macho_info->module->format_info[DFI_MACHO]->u.macho_info->in_use = 1;
......@@ -1480,7 +1476,7 @@ struct module* macho_load_module(struct process* pcs, const WCHAR* name, unsign
return NULL;
}
BOOL macho_load_debug_info(struct module* module, struct macho_file_map* fmap)
BOOL macho_load_debug_info(struct module* module)
{
return FALSE;
}
......
......@@ -366,7 +366,7 @@ BOOL module_get_debug(struct module_pair* pair)
&idslW64);
break;
case DMT_MACHO:
ret = macho_load_debug_info(pair->effective, NULL);
ret = macho_load_debug_info(pair->effective);
break;
default:
ret = FALSE;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment