mirror of
https://github.com/reactos/reactos.git
synced 2024-09-30 22:47:28 +00:00
[0.4.7][WIN32SS] Remove an ambiguous assert entirely CORE-16738
It popped up very reliably when performing git-clone of the ros sources while having the taskmgr open in the processes-tab. Or when building 'ninja bootcd -j1' while having the taskmgr open in the processes-tab. It is always possible to ignore it and then continue using ros for days without noticing any side effects. James Tabor judged the assert to have a "questionable logic". Giannis who once added it, was also ok with commenting it. His words: This assertion isn't something fatal, the worst thing that can happen from continuing on it would be to make the thread get awake more times than it should. The assert was once added by SVN r63880 == git9ad857369a
Fix picked from: 0.4.14-dev-1519-g87f6c82d85
commenting 2020-04-21 0.4.15-dev-3780-gbf16435303
removal 2022-02-05
This commit is contained in:
parent
f81f89e9a6
commit
0a17dd8a99
|
@ -395,7 +395,7 @@ NtUserGetGUIThreadInfo(
|
|||
EngSetLastError(ERROR_ACCESS_DENIED);
|
||||
RETURN( FALSE);
|
||||
}
|
||||
|
||||
|
||||
if ( W32Thread->MessageQueue )
|
||||
MsgQueue = W32Thread->MessageQueue;
|
||||
else
|
||||
|
@ -703,8 +703,6 @@ void UserDbgAssertThreadInfo(BOOL showCaller)
|
|||
ASSERT(pci->ulClientDelta == DesktopHeapGetUserDelta());
|
||||
if (pti->pcti && pci->pDeskInfo)
|
||||
ASSERT(pci->pClientThreadInfo == (PVOID)((ULONG_PTR)pti->pcti - pci->ulClientDelta));
|
||||
if (pti->pcti && IsListEmpty(&pti->SentMessagesListHead))
|
||||
ASSERT((pti->pcti->fsChangeBits & QS_SENDMESSAGE) == 0);
|
||||
if (pti->KeyboardLayout)
|
||||
ASSERT(pci->hKL == pti->KeyboardLayout->hkl);
|
||||
if(pti->rpdesk != NULL)
|
||||
|
|
Loading…
Reference in a new issue