mboost-dp1

To -Werror or not


Gå til bund
Gravatar #2 - larsp
8. sep. 2021 15:14
-Werror har reddet mig mange gange. Kører altid med det slået til. Men Linux kernel projektet og alle dets konfiguration er er selvfølgelig et helt andet dyr, så held og lykke til de maintainere der skal rydde op i det nu.

Jeg er fornylig stødt på en patch hvor en gut prøver at fixe en hash funktion der bruger en switch case fallthrough konstruktion, som fører til warnings i nyere byggemiljøer, og fixet var at indsætte breaks! (Det korrekte fix er /* fallthrough */ kommentarer). Det er et (sjældent) eksempel på at -Werror fører til introduktion af fejl.

P.S. Her er patchen: http://git.yoctoproject.org/cgit/cgit.cgi/meta-clo...
Gravatar #3 - larsp
8. sep. 2021 15:28
P.P.S. Min løsning var at bygge med en nyere version af uwsgi sourcen hvor de for længst har løst dette fallthrough problem, i stedet for førnævnte recipe der kører med en ældre source + buggy patches.
Gravatar #4 - arne_v
8. sep. 2021 15:50
#substans

-Werror er vel et forsøg på en teknisk løsning på et ikke-teknisk problem.

Den store beslutning er at man ikke vil leve med warnings.

Med en velfungerende organisation går folk i gang med at fjerne warnings.

I en ikke velfungerende organisation ignoreres beslutningen. Og man kan så sætte tommeskruerne på med -Werror fordi så har de pludseligt ingen output.
Gravatar #5 - arne_v
8. sep. 2021 15:53
#2 og #3

Fallthrough switch burde være en syntax fejl.

C# fik gjordt det rigtigt.

Fordelene er så minimale i forhold til risikoen for utilsigtet fallthrough.

Gå til top

Opret dig som bruger i dag

Det er gratis, og du binder dig ikke til noget.

Når du er oprettet som bruger, får du adgang til en lang række af sidens andre muligheder, såsom at udforme siden efter eget ønske og deltage i diskussionerne.

Opret Bruger Login