Didn't know that BView::do_owner_check() also checks the lock - maybe
it should be renamed to do_owner_and_lock_check(). Removed looper locking again, since it's not needed, then. Thanks to Adi for the hint. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@12743 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
@@ -1205,10 +1205,6 @@ BView::GetMouse(BPoint *location, uint32 *buttons, bool checkMessageQueue)
|
|||||||
queue->Unlock();
|
queue->Unlock();
|
||||||
}
|
}
|
||||||
|
|
||||||
// Since we're usually called locked, this is mainly needed
|
|
||||||
// in case we were called from another thread
|
|
||||||
LockLooper();
|
|
||||||
|
|
||||||
// If no mouse update message has been found in the message queue,
|
// If no mouse update message has been found in the message queue,
|
||||||
// we get the current mouse location and buttons from the app_server
|
// we get the current mouse location and buttons from the app_server
|
||||||
|
|
||||||
@@ -1221,8 +1217,6 @@ BView::GetMouse(BPoint *location, uint32 *buttons, bool checkMessageQueue)
|
|||||||
owner->fLink->Read<BPoint>(location);
|
owner->fLink->Read<BPoint>(location);
|
||||||
owner->fLink->Read((int32 *)buttons, sizeof(int32));
|
owner->fLink->Read((int32 *)buttons, sizeof(int32));
|
||||||
}
|
}
|
||||||
|
|
||||||
UnlockLooper();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user