You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hmm... per the gcc docs, "attribute ((__const))" isn't really supposed to be legal... probably an accident of keyword mapping. A patch which special-cases __const would be better, because gcc doesn't accept, for example, "attribute((__noreturn))".
fixed by r116570.
Contains a comment. The fix is so simple, that a test for it is (IMHO) not needed, anyway our app tests it well, so I'll notice when somebody breaks it :-)
Extended Description
We seem to have a legacy header (and no process in place to change it) which looks pretty much like this:
http://en.wikibooks.org/wiki/C++_Programming/ctype.h_header
Clang warns on the "attribute ((__const))" part:
warning: unknown attribute '__const' ignored [-Wunknown-attributes]
attribute ((__const));
^
I propose to add '__const' as an attribute, becoming synonymous to 'const'. I am about to make a patch and shall attach it here.
The text was updated successfully, but these errors were encountered: