search

lms-org / config_auto_drive

main repository for config files (includes all needed modules & libs as git submodules)
1 stars 0 forks source link

Wer liest tatsächlich noch Compiler Warnings? #81

Closed art-mth closed 7 years ago

art-mth commented 7 years ago

Bei jedem mal kompilieren wird meine Kommandozeile mit Compiler warnings zugeschüttet, sodass es manchmal schwer ist error zu finden. Ich bin grundsätzlich ein Fan von Compiler warnings, aber in diesem Fall haben sie meiner Meinung nach negativen Wert. Es es sei denn jemand sagt er liest und fixt aktiv diese Compiler Warnings würde ich stark dafür voten sie auszuschalten. Das würde dann bedeuten peu a peu wenn sie einem auf den Sack gehen die flags (Wall etc) zu entfernen. Oder aber wir veranstalten ein kleines Fix-It wo wir uns gemeinsam (natürlich time boxed) Zeit nehmen die Dinger los zu werden. Was meint ihr?

syxolk commented 7 years ago

Die Fehler im Framework kommen vor allem von den generierten Protobuf-Dateien, bis vor ein paar Monaten war da nicht eine einzige Warning. Ansonsten gibt es meist Unused Parameter Probleme, die sollen natürlich weg.

Ich würde Warnings ungern ausschalten, wir haben damals -Wall -Wextra -Wpedantic aus gutem Grund aktiviert. Wir können uns aber gern mal zusammensetzen und das gröbste fixen.