Give a more informative error when a DLL fails to initialize

Robert Shearman rob at codeweavers.com
Mon Jan 10 11:31:21 CST 2005


Mike Hearn wrote:
...

> 
> static NTSTATUS load_dll( LPCWSTR load_path, LPCWSTR libname, DWORD flags, WINE_MODREF** pwm );
> static FARPROC find_named_export( HMODULE module, const IMAGE_EXPORT_DIRECTORY *exports,
>@@ -893,9 +894,17 @@ static NTSTATUS process_attach( WINE_MOD
>         WINE_MODREF *prev = current_modref;
>         current_modref = wm;
>         if (MODULE_InitDLL( wm, DLL_PROCESS_ATTACH, lpReserved ))
>+        {
>             wm->ldr.Flags |= LDR_PROCESS_ATTACHED;
>+        }
>  
>

You don't need to add these unnecessary brackets. They add nothing to 
the readability of the code.

Rob



More information about the wine-devel mailing list