[Webkit-unassigned] [Bug 137485] [GLIB] Split GMainLoopSource moving thread safe implementation to its own class GThreadSafeMainLoopSource

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=137485


Sergio Villar Senin <svillar at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #239411|review?                     |review+, commit-queue-
               Flag|                            |




--- Comment #14 from Sergio Villar Senin <svillar at igalia.com>  2014-10-10 03:53:04 PST ---
(From update of attachment 239411)
View in context: https://bugs.webkit.org/attachment.cgi?id=239411&action=review

>>>> Source/WTF/wtf/gobject/GMutexLocker.h:78
>>>> +            return;
>>> 
>>> Hmm this makes the template useless, because the recursive mutexes will act as normal ones, won't they?
>> 
>> Hmm this makes the template useless, because the recursive mutexes will act as normal ones, won't they?
> 
> The idea is that the same GMutexLocker object shouldn't lock a mutex already locked, but two different GMutexLocker objects, wrapping the same GRecMutex should be allowed. Which is the case I'm covering here, schedule calls cancel, both locking the mutex, but using different GMutexLocker objects.

Right. Too many mutexes and lockers :). Couldn't we achieve the same by using the _try_ variants in MutexWrapper object?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.