After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 730293 - clang++-3.4: error: 'register' storage class specifier is deprecated
clang++-3.4: error: 'register' storage class specifier is deprecated
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: build
2.38.x
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2014-05-17 06:25 UTC by Jan Kratochvil
Modified: 2014-06-28 17:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
all: remove use of 'register' keyword (40.47 KB, patch)
2014-05-25 10:19 UTC, Allison Karlitskaya (desrt)
committed Details | Review

Description Jan Kratochvil 2014-05-17 06:25:45 UTC
echo '#include <glib.h>'|clang++ -c $(pkg-config --cflags glib-2.0) -Werror -std=c++11 -x c++ -
In file included from <stdin>:1:
In file included from /usr/include/glib-2.0/glib.h:56:
In file included from /usr/include/glib-2.0/glib/giochannel.h:36:
In file included from /usr/include/glib-2.0/glib/gstring.h:37:
/usr/include/glib-2.0/glib/gutils.h:321:3: error: 'register' storage class specifier is deprecated
      [-Werror,-Wdeprecated-register]
  register guint n_bits = 0;
  ^~~~~~~~~
1 error generated.

Fedora 20 x86_64
glib2-devel-2.38.2-2.fc20.x86_64
clang-3.4-6.fc20.x86_64
Comment 1 Allison Karlitskaya (desrt) 2014-05-25 10:19:32 UTC
Created attachment 277140 [details] [review]
all: remove use of 'register' keyword

We should have done this a decade ago...



Does anyone have any argument against this?  In particular, it seems a
bit strange to do it to pcre considering this is a copy-paste lib, more
or less...
Comment 2 Dan Winship 2014-06-07 13:03:37 UTC
Comment on attachment 277140 [details] [review]
all: remove use of 'register' keyword

yay

> Does anyone have any argument against this?  In particular, it seems a
> bit strange to do it to pcre considering this is a copy-paste lib, more
> or less...

we appear to already be making lots of changes to the internal copy of pcre. eg: https://git.gnome.org/browse/glib/commit/?id=706b72db
Comment 3 Matthias Clasen 2014-06-28 17:36:23 UTC
Attachment 277140 [details] pushed as dce8876 - all: remove use of 'register' keyword