On Monday, August 27, 2012 09:17:43 Thiago Macieira wrote: > On domingo, 26 de agosto de 2012 19.09.15, Vadim Zhukov wrote: > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > http://git.reviewboard.kde.org/r/106224/ > > ----------------------------------------------------------- > > > > Review request for kdelibs and Michael Pyne. > > > > > > Description > > ------- > > > > Add simple spin locking mechanism. > > Written by Michael Pyne as from > > https://bugs.kde.org/attachment.cgi?id=73282 , with some tweaking by me. > > > > > > This addresses bug 305023. > > > > http://bugs.kde.org/show_bug.cgi?id=305023 > > Please use the Qt atomic types. Until GCC 4.7, they generate better code.
I agree, the reason it wasn't that way initially is mentioned in the discussion on the bug (but basically because you can't simply put QBasicAtomicInt in the union used to store the different lock types that are possible). I have an idea of what to do to fix that but haven't had time to code it up. Regards, - Michael Pyne
signature.asc
Description: This is a digitally signed message part.