diff options
| author | Philip Withnall <pwithnall@endlessos.org> | 2020-11-11 18:38:36 +0000 |
|---|---|---|
| committer | Philip Withnall <pwithnall@endlessos.org> | 2020-11-20 14:40:19 +0000 |
| commit | 9474655eb21d64519b293e780bb686976cbdb790 (patch) | |
| tree | a24ad3bee6cedcab1526f3adc6204317cbc33105 | |
| parent | 3c648457c284b4ba313b8591008d2e18ae4335eb (diff) | |
| download | glib-9474655eb21d64519b293e780bb686976cbdb790.tar.gz | |
gatomic: Drop unnecessary volatile qualifiers from macro variables
It’s not necessary and provides no thread safety guarantees.
The `volatile` qualifiers on the function arguments have to be kept, as
they are (unfortunately) part of the API.
Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
Helps: #600
| -rw-r--r-- | glib/gatomic.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/glib/gatomic.h b/glib/gatomic.h index bb1435c70..e6eccfada 100644 --- a/glib/gatomic.h +++ b/glib/gatomic.h @@ -211,7 +211,7 @@ G_END_DECLS })) #define g_atomic_pointer_and(atomic, val) \ (G_GNUC_EXTENSION ({ \ - volatile gsize *gapa_atomic = (volatile gsize *) (atomic); \ + gsize *gapa_atomic = (gsize *) (atomic); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gpointer)); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gsize)); \ (void) (0 ? (gpointer) *(atomic) : NULL); \ @@ -220,7 +220,7 @@ G_END_DECLS })) #define g_atomic_pointer_or(atomic, val) \ (G_GNUC_EXTENSION ({ \ - volatile gsize *gapo_atomic = (volatile gsize *) (atomic); \ + gsize *gapo_atomic = (gsize *) (atomic); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gpointer)); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gsize)); \ (void) (0 ? (gpointer) *(atomic) : NULL); \ @@ -229,7 +229,7 @@ G_END_DECLS })) #define g_atomic_pointer_xor(atomic, val) \ (G_GNUC_EXTENSION ({ \ - volatile gsize *gapx_atomic = (volatile gsize *) (atomic); \ + gsize *gapx_atomic = (gsize *) (atomic); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gpointer)); \ G_STATIC_ASSERT (sizeof *(atomic) == sizeof (gsize)); \ (void) (0 ? (gpointer) *(atomic) : NULL); \ |