search

voikko / corevoikko

Libvoikko and essential linguistic resources
Other
89 stars 25 forks source link

unused params and Werror #11

Closed flammie closed 10 years ago

flammie commented 10 years ago 
$ g++ --version
g++ (Ubuntu 4.8.2-19ubuntu1) 4.8.2
$ make
[...]
grammar/NullGrammarChecker.cpp:36:7: error: unused parameter ‘token’ [-Werror=unused-parameter]
  void analyseToken(Token * token) {
       ^
grammar/NullGrammarChecker.cpp:39:13: error: unused parameter ‘text’ [-Werror=unused-parameter]
  Sentence * analyseSentence(const wchar_t * text, size_t textlen, size_t sentencepos) {
             ^
grammar/NullGrammarChecker.cpp:39:13: error: unused parameter ‘textlen’ [-Werror=unused-parameter]
grammar/NullGrammarChecker.cpp:39:13: error: unused parameter ‘sentencepos’ [-Werror=unused-parameter]
grammar/NullGrammarChecker.cpp:43:14: error: unused parameter ‘text’ [-Werror=unused-parameter]
  Paragraph * analyseParagraph(const wchar_t * text, size_t textlen) {
              ^
grammar/NullGrammarChecker.cpp:43:14: error: unused parameter ‘textlen’ [-Werror=unused-parameter]
grammar/NullGrammarChecker.cpp:50:7: error: unused parameter ‘para’ [-Werror=unused-parameter]
  void check(const Paragraph * para) {
       ^
cc1plus: all warnings being treated as errors
make[3]: *** [grammar/NullGrammarChecker.lo] Virhe 1
make[3]: Poistutaan hakemistosta "/home/tpirinen/Koodit/corevoikko/libvoikko/src"
make[2]: *** [all] Virhe 2
make[2]: Poistutaan hakemistosta "/home/tpirinen/Koodit/corevoikko/libvoikko/src"
make[1]: *** [all-recursive] Virhe 1
make[1]: Poistutaan hakemistosta "/home/tpirinen/Koodit/corevoikko/libvoikko"
make: *** [all] Virhe 2

Consider dropping Werror from default configurations and adding it to your personal environments FLAGS variables.