search

jonnydee / nzmqt

nzmqt - A lightweight C++ Qt binding for ZeroMQ
Other
199 stars 72 forks source link

warnings with -Wall (and compilation broken if -Werror) #12

Closed wieckom closed 11 years ago

wieckom commented 11 years ago

We compile everything with -Wall -Werror.

With current git version of nzmqt it's not possible - there are commas at the end of enums, which lead to

nzmqt-3.2.0-dev/include/nzmqt/nzmqt.hpp:152:32: warning: comma at end of enumerator list [-pedantic] nzmqt-3.2.0-dev/include/nzmqt/nzmqt.hpp:159:38: warning: comma at end of enumerator list [-pedantic] nzmqt-3.2.0-dev/include/nzmqt/nzmqt.hpp:166:39: warning: comma at end of enumerator list [-pedantic] nzmqt-3.2.0-dev/include/nzmqt/nzmqt.hpp:172:39: warning: comma at end of enumerator list [-pedantic] nzmqt-3.2.0-dev/include/nzmqt/nzmqt.hpp:198:38: warning: comma at end of enumerator list [-pedantic]

Simply removing the last commas in the 5 enums [or 6 if you fix the commented-out one too] fixes it.

$ g++ --version g++ (GCC) 4.7.1