<div dir="ltr">On Fri, Jun 26, 2015 at 6:48 AM, Steve Lhomme <span dir="ltr"><<a href="mailto:robux4@videolabs.io" target="_blank">robux4@videolabs.io</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Jun 26, 2015 at 11:29 AM, Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br>
> Le 2015-06-26 11:32, Steve Lhomme a écrit :<br>
>><br>
>> On Thu, Jun 25, 2015 at 11:09 PM, Rémi Denis-Courmont<br>
</span>>> <<a href="mailto:git@videolan.org">git@videolan.org</a>> wrote:<br>
>>><br>
>>> vlc | branch: master | Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> | Fri Jun 26<br>
<span class="">>>> 00:06:44 2015 +0300| [d2571e643edb0f1cb217805ef8d5ac172d59a864] | committer:<br>
>>> Rémi Denis-Courmont<br>
>>><br>
>>> qt: use QAtomicInt<br>
>>><br>
>>>><br>
>>>><br>
>>>> <a href="http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=d2571e643edb0f1cb217805ef8d5ac172d59a864" rel="noreferrer" target="_blank">http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=d2571e643edb0f1cb217805ef8d5ac172d59a864</a><br>
>>><br>
</span><span class="">>>> ---<br>
>>><br>
>>>  modules/gui/qt4/dialogs/messages.cpp |    4 ++--<br>
>>>  modules/gui/qt4/dialogs/messages.hpp |    4 ++--<br>
>>>  2 files changed, 4 insertions(+), 4 deletions(-)<br>
>>><br>
>>> diff --git a/modules/gui/qt4/dialogs/messages.cpp<br>
>>> b/modules/gui/qt4/dialogs/messages.cpp<br>
>>> index 21c1110..d23007f 100644<br>
>>> --- a/modules/gui/qt4/dialogs/messages.cpp<br>
>>> +++ b/modules/gui/qt4/dialogs/messages.cpp<br>
>>> @@ -146,7 +146,7 @@ MessagesDialog::~MessagesDialog()<br>
>>><br>
>>>  void MessagesDialog::changeVerbosity( int i_verbosity )<br>
>>>  {<br>
>>> -    atomic_store( &this->verbosity, i_verbosity );<br>
>>> +    verbosity = i_verbosity;<br>
>>>  }<br>
>>><br>
>>>  void MessagesDialog::updateConfig()<br>
>>> @@ -340,7 +340,7 @@ void MessagesDialog::MsgCallback( void *self, int<br>
>>> type, const vlc_log_t *item,<br>
>>>  {<br>
>>>      MessagesDialog *dialog = (MessagesDialog *)self;<br>
>>>      char *str;<br>
>>> -    int verbosity = atomic_load( &dialog->verbosity );<br>
>>> +    int verbosity = dialog->verbosity;<br>
>><br>
>><br>
>> This fails to build for me in msys2+mingw64.<br>
>> `error: cannot convert 'QAtomicInt' to 'int' in initialization`<br>
><br>
><br>
> According to Qt doc, there is an operator int. It is equivalent to explicit<br>
> load, AFAIK.<br></span></blockquote><div><br></div><div>This was added in Qt 5.3, so earlier versions fail to build.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br>
</span>I tried that first. But it didn't work:<br>
error: expected unqualified-id before 'int'<br>
<br>
It may have to do with the fact MsgCallback is a static callback.<br>
<span class="im HOEnZb"><br>
> --<br>
> Rémi Denis-Courmont<br>
> <a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
</span><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br></div></div>