certi-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [certi-devel] Patch GAV_aux.cc


From: Valéry Raulet
Subject: Re: [certi-devel] Patch GAV_aux.cc
Date: Wed, 27 Nov 2002 15:33:44 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b) Gecko/20021016

Benoît Bréholée wrote:

Valéry Raulet <address@hidden> writes:

-> ajout de inline dans les fonctions avec seulement un return

Certains optimiseurs devraient le faire automatiquement, mais ça ne
coute rien de le mettre explicitement.

-> changement de la recopie for(unsigned int i=0; ...) buff[i] =
tmp->_value[i]; par un memcpy()

Il y avait eu des problèmes à cause de l'emploi de strcpy (qui
s'arrête dès qu'on rencontre un 0), mais memcpy devrait effectivement
faire la même chose, et plus rapidement.

-> résolution de deux types de fuites mémoire :
        -> lors d'un remove d'AttributeHandleSet, ...
        -> lors d'un empty d'AttributeHandleSet, ...

J'avais commencé à modifier les destructeurs pour profiter de ces
corrections, mais je viens de voir que c'est au programme du patch
suivant :) Je commite déjà ça. Pour les autres patches, je n'arrive
pas à récupérer le #732, apparemment il était gzippé ? Je pense que le
patch manager est plus adapté à du texte brut pour pouvoir lire
facilement en ligne.
Toujours à propos des patches, mieux vaut si possible utiliser
exclusivement le Patch Manager plutot que faire des attachements :
comme la liste est archivée sur mail.gnu.org, autant ne pas trop
l'encombrer puisque le patch sera toujours accessible soit dans le CVS
s'il est appliqué, soit dans le Patch Manager s'il est en attente.

Merci pour ces corrections, je passe aux patches suivants...


Le patch #732 correspond à ce mail etle mail suivant (GAV_aux suite...). J'ai vu ce matin qu'on pouvait soumettre des patchs !
Maintenant, je le ferai comme ça.

Par contre pour la mise en forme du code source, y a t-il des conventions d'écriture ? (ca serait bien pour homogènéiser l'ensemble de sources)

Autre question : vous utilisez peu les fonctionnalités de la STL. Je voulais savoir si c'était un choix imposé ?

       Valéry.






reply via email to

[Prev in Thread] Current Thread [Next in Thread]