[grisbi-devel] [PATCH 0/7] Bug fix and various memory improvements in payees tab

[Rémi Cardona]

Salut à tous,

J'ai proposé à Pierre, il y a quelques jours, que nous essayions de
mettre en place de la relecture de code a priori. Loin de vouloir
alourdir notre travail bénévole par la bureaucratie, voici certains de
ses bénéfices.

Tout d'abord, augmenter la qualité des commits. Il n'est pas rare (et
j'en suis également coupable) de voir certains commits se faire suivre
de petits correctifs réparant un printf() oublié, un free() égaré ou
plus simplement une faute d'orthographe dans un message. Une relecture
des patches sur la liste de diffusion permettrait à d'autres de repérer
ces coquilles et de demander à l'auteur du patch de soumettre une
version corrigée.

Un deuxième bénéfice est de nous inciter à nous poser la question « ce
que j'écris est-il lisible par quelqu'un d'autre? ». Savoir qu'une autre
paire d'yeux relira ce code a généralement pour effet de "proprifier",
nettoyer, séparer ses commits afin qu'ils soient plus faciles à relire.

Enfin le dernier bénéfice est tout simplement d'apprendre des autres.
Apprendre du code des autres, mais aussi des discussions qui peuvent
survenir lors des relectures. Grisbi est un projet de taille conséquente
(178 000 lignes de code) dont aucune personne ne peut connaître tous les
recoins. La lecture d'un patch, aussi rapide soit-elle, peut permettre
au relecteur de découvrir d'autres aspects de Grisbi.

Plutôt que de continuer mon beau discours, voici une première série de
patches concernant le panneau de tiers et l'assistant de fusion de
tiers:

 - le premier supprime du code inutile (la plupart des fonctions/macros
   g_object/G_OBJECT sont assez coûteuses)
 - le deuxième corrige un vrai bug de l'interface, facilement
   reproductible
 - le troisième supprime une destruction/création d'un GtkListStore
   (mais il n'y avait pas de fuite de mémoire, juste du travail inutile)
 - les 4 derniers corrigent de multiples fuites de mémoire (rien de bien
   grave, mais c'est toujours ça en moins)

Voila, n'hésitez pas à me dire ce que vous en pensez, à me poser des
questions. 

Si personne n'a d'objection, je pousserai ces patches d'ici quelques
jours dans la branche 1.0.x.

Rémi