Call the user signal proc for exe modules too, to avoid duplicating

dlls/kernel/kernel32.spec

@@ -1088,7 +1088,6 @@
 @ stdcall GetModuleFileName16(long ptr long)
 @ stdcall GetModuleHandle16(str)
 @ stdcall GetModuleName16(long ptr long)
-@ stdcall GetModuleUsage16(long)
 @ stdcall GetSelectorBase(long)
 @ stdcall GetSelectorLimit16(long)
 @ stdcall GetThreadQueue16(long)

dlls/user/user_main.c

@@ -292,21 +292,6 @@ static void thread_detach(void)
     WIN_DestroyThreadWindows( GetDesktopWindow() );
     QUEUE_DeleteMsgQueue();
 
-    if (!(NtCurrentTeb()->tibflags & TEBF_WIN32))
-    {
-        HMODULE16 hModule = GetExePtr( MapHModuleLS(0) );
-
-        /* FIXME: maybe destroy menus (Windows only complains about them
-         * but does nothing);
-         */
-        if (GetModuleUsage16( hModule ) <= 1)
-        {
-            /* ModuleUnload() in "Internals" */
-            /* HOOK_FreeModuleHooks( hModule ); */
-            CLASS_FreeModuleClasses( hModule );
-            CURSORICON_FreeModuleIcons( hModule );
-        }
-    }
     exiting_thread_id = 0;
 }
 

loader/ne/module.c

@@ -1282,13 +1282,11 @@ static BOOL16 NE_FreeModule( HMODULE16 hModule, BOOL call_wep )
 
     if (call_wep && !(pModule->flags & NE_FFLAGS_WIN32))
     {
+        /* Free the objects owned by the DLL module */
+        NE_CallUserSignalProc( hModule, USIG16_DLL_UNLOAD );
+
         if (pModule->flags & NE_FFLAGS_LIBMODULE)
-        {
             MODULE_CallWEP( hModule );
-
-            /* Free the objects owned by the DLL module */
-            NE_CallUserSignalProc( hModule, USIG16_DLL_UNLOAD );
-        }
         else
             call_wep = FALSE;  /* We are freeing a task -> no more WEPs */
     }