synchronized (this[.classinfo]) in druntime and phobos

Alex Rønne Petersen alex at lycus.org
Tue May 29 10:51:58 PDT 2012


On 29-05-2012 19:37, Jess wrote:
> On Tuesday, 29 May 2012 at 17:01:01 UTC, Alex Rønne Petersen wrote:
>>>>
>>>
>>> druntime is the low-level runtime library of D. This discussion is about
>>> high-level class design.
>>>
>>
>> I should also add that the reason I started this thread was to prevent
>> common locking errors induced by the anti-patterns synchronized
>> causes. If you're calling *anything* in the core.memory module, I sure
>> hope you know what you're doing in general.
>>
>> I really don't think these two issues are connected at all.
>
> ok, maybe I read too much into your statement below? it sounded as if
> you wanted to ban this usage from druntime?
>
> Alex Rønne Petersen wrote:
> "I've seen several occurrences of synchronized (this) and synchronized
> (this.classinfo) in both druntime and phobos by now. I propose that we
> officially ban these patterns with extreme prejudice."
>

Yes, I do. But I'm not sure what that has to do with my GC.callLocked() PR?

-- 
Alex Rønne Petersen
alex at lycus.org
http://lycus.org


More information about the Digitalmars-d mailing list