Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
Benjamin Chrétien
@bchretien
il le stock
qui gère la mémoire ?
Stanislas Brossette
@stanislas-brossette
Et c'est lié à LA fuite mémoire que j'arrive pas à pluguer
Benjamin Chrétien
@bchretien
pourquoi garder un raw pointer ? si t'utilises fct() pour la récupérer
Stanislas Brossette
@stanislas-brossette
Car trop de repercussions dans tous les sens
Je sais pas pourquoi il fait ça.
Benjamin Chrétien
@bchretien
ca a été fait comme ca car aucune considération pour la gestion de la mémoire, mais maintenant que c'est quasiment réglé en interne, il reste à corriger cette classe là qui est le point d'entrée de l'utilisateur
si on ne veut pas casser l'API
on peut faire ce que j'ai fait pour roboptim-core
Stanislas Brossette
@stanislas-brossette
CàD?
jusque là une const ref était stockée, maintenant un shared_ptr est pris en entrée (donc similaire aux méthodes d'ajout de contraintes)
la version const ref est toujours disponible (bien que deprecated)
en interne la fonction cout est stockée comme un shared_ptr
et si une const ref est passée, le shared_ptr est initialisé sans deleter
comme ca on garde le même comportement
Stanislas Brossette
@stanislas-brossette
Mouais.
Benjamin Chrétien
@bchretien
(i.e. crash si la fonction coût est delete)
Stanislas Brossette
@stanislas-brossette
Ah oui ca c'est bien
J'vais faire un essai
Benjamin Chrétien
@bchretien
et l'utilisateur se prend un message de deprecated là où il utilise encore le constructeur unsafe
Stanislas Brossette
@stanislas-brossette
A voir pour le message
Ca empeche de compiler en mode nazi
Benjamin Chrétien
@bchretien
bah tu as juste à flag le constructeur comme deprecated
justement, là t'es bien content que ton compilateur te dise qu'il vaudrait mieux corriger ton code ;)
Stanislas Brossette
@stanislas-brossette
Oui, mais on pourra plus compiler les codes qui l'utilise en -werror
Benjamin Chrétien
@bchretien
soit tu désactives -Werror, soit tu corriges le code
Stanislas Brossette
@stanislas-brossette
C'est ca le probleme
Benjamin Chrétien
@bchretien
tu dois aussi pouvoir passer -Wno-deprecated
Stanislas Brossette
@stanislas-brossette
Je voudrais continuer à utiliser werror malgré l'existence et l'utilisationde cette methode
Benjamin Chrétien
@bchretien
   -Wno-deprecated
       Do not warn about usage of deprecated features.
   -Wno-deprecated-declarations
       Do not warn about uses of functions, variables, and types marked as deprecated by using the "deprecated" attribute.
Stanislas Brossette
@stanislas-brossette
Ca doit être possible d'ignorer l'aspect deprecated juste pour une fonction
Benjamin Chrétien
@bchretien
mais là ca se corrige vite dans le code utilisateur, j'ai fait ca pour mon code avec le changement dans roboptim-core
Stanislas Brossette
@stanislas-brossette
Parceque si autrechose est deprecated, on veut le savoir
Oui, mais Roboptim-core-manifold ne compilera pas
Benjamin Chrétien
@bchretien
pour autant que je sache tu ne peux pas ignorer le deprecated sur une fonction spécifique sans modifier le code
Stanislas Brossette
@stanislas-brossette
Ouais, après on sait que c'est la seule dans core-manifold, mais bon
D'ailleurs std ou boost les shared_ptr dans DescWrapper?
Benjamin Chrétien
@bchretien
hmmm bonne question
dans mon code j'utilise déjà des boost::shared_ptr à cet endroit là
Stanislas Brossette
@stanislas-brossette
Et moi des std
Benjamin Chrétien
@bchretien
après rien n'empêche de supporter les 2
Stanislas Brossette
@stanislas-brossette
true
Benjamin Chrétien
@bchretien
tu stock en std::shared_ptr en interne
et si tu recois un boost::shared_ptr, tu fais la conversion
Stanislas Brossette
@stanislas-brossette
Deal
Benjamin Chrétien
@bchretien
comme ca, tout le monde est content