[Github-comments] [geany/geany] Don't use volatile (#2907)

[elextr]

Well I was more worried about gcc 7 than clang 12 but I see Travis is using gcc 7.5 so probably ok.

> Ah, as I said - on latest main branch g_once_init_enter still has arg qualified by volatile qualifier right after asking not to use volatile.

Ahh well, who knows what they are thinking, but anyway as my edit said it should be fine to pass a non-volatile to a volatile.  And as I noted below, its a macro, not a real function, so parameter types are not checked.

> (but now I'm puzzled why this warning then? Maybe I'm missing something)

Well, all the "functions" are actually macros (hisss booo) according to the error messages in the OP, until the _atomic load() function, so the type from the user code gets passed through to that function which does not have "volatile" on its prototype.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2907#issuecomment-927627433
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210927/c07b2193/attachment.htm>