[grisbi-devel] Grisbi compiler warnings and bugs

[Pierre Biava]

Ludovic Rousseau a écrit le 06/10/2016 à 20:44 :

Bonsoir Ludovic,

>
> Je commence à corriger les warnings. Juste avec clang et le niveau de 
> warnings de base j'ai déjà plusieurs warnings inquiétants.
>
> Par exemple j'en ai corrigé 1 qui est aussi un bug :
> https://github.com/grisbi/grisbi/commit/f28fd44812a9112c670568849ae491a73f5e7d4e
>
Des comme ça il doit y en avoir d'autres. Si on peut systématiquement 
les trouver c'est bien. Jusqu'à présent seul le hasard ou si ça 
conduisait à un bug effectif nous les faisait corriger. Après est-ce 
inquiétant peut-être pas. Ça doit plutôt nous faire réfléchir sur la 
pertinence du code ou la
nécessité de la fonction.
> Mon idée est de corriger tous les warnings et ensuite de configurer 
> Travis-CI avec -Werror pour qu'un nouveau warning fasse échouer la 
> compilation automatique.

Ce serait bien. d'y arriver. Toutefois faire attention au type de 
warning. Il est difficile de ne plus en avoir.

En parlant de warning j'ai une question par rapport à gcc. Ça fait 
longtemps que gcc ne m'alerte plus sur les variables inutilisées. J'ai 
testé plusieurs options mais aucune ne me rend cette erreur. Si tu avais 
une idée pour réparer ça ce serait bien de modifier le configure.ac pour 
que ça refonctionne.

Bonne soirée.

-- 

A+

Pierre Biava