Re: [certi-devel] Patch 754 & 755

[Benoît Bréholée]

Valéry Raulet <address@hidden> writes:

> >- pour la classe list STL : pas de problème, mais il faudrait partir
> >  d'une version bien déterminée, j'applique ces patches dès que
> >  possible et je marque un tag pour faire une version de développement.
> >
> OK, tu me dis dès que c'est possible.

Voila : CERTI-3_0a

Je n'ai pas appliqué les patches : le 754 fait planter le RTIG
(j'aurais du commencer par tester ça!) dès qu'il y a échange de
données (juste après avoir fait Entrée dans le billard). Je le laisse
en attente pour l'instant. Et donc je laisse le 755 aussi en attente
comme il y avait des problèmes dessus, + remarque ci-dessous.

> >>Corrections de types : int ou unsigned int en UShort ou ULong
>
> OK mais on peut quand même utiliser ulong, ushort (gcc les comprends)
> pour garder le code moins long ?

J'ai trouvé des points de vue contradictoires avec ces
utilisations. Je ne suis pas sûr que ce soit une bonne chose de le
remplacer par un short et/ou de le mettre en unsigned quand ça ne
vient pas d'un véritable besoin sur la donnée. Surtout si on tient
compte de la lisibilité du code. Tu vois vraiment des avantages ou
c'est plus une question de style de codage ?
Pour l'instant ce que j'ai retenu contre cette utilisation :
- le unsigned n'apporte pas vraiment de garanties que l'entier sera
  positif (donc ça complique si ça ne sert qu'à ça)
- int est plus simple pour la lisibilité (ça compte quand meme)
- le short est a priori moins simple à utiliser pour le processeur, la
  place gagnée n'est souvent pertinente que si on veut faire des
  structures de données à sauvegarder sur le plus petit espace
  possible.

Sur les premiers patches je pensais que c'était préférable à cause de
certains compilateurs, mais apparemment ça ne pose pas de problèmes...

Je vais essayer de voir le pour et le contre, avoir quelques avis, et
puis si c'est effectivement préférable, ce sera à appliquer plus
globalement (parce que là ce serait à détailler dans chaque
commentaire CVS ou entrée de ChangeLog où il y en a!)


-- Benoit