Skip to content

Fix warnings - #393

Open
Jordan08 wants to merge 3 commits into
codac-team:codac2from
Jordan08:fix-warnings
Open

Fix warnings#393
Jordan08 wants to merge 3 commits into
codac-team:codac2from
Jordan08:fix-warnings

Conversation

@Jordan08

Copy link
Copy Markdown

J'ai développé avec Clang-18 et plein d'option de Debug et de track de memory leak
ça a révelé beaucoup de warnings et de perte de memoire.

J'ai fixé l'intégralité de ce que j'ai trouvé.

  • j'ai ajouté plein de check et de warning dans le mode Debug
  • quelques problèmes de typage dans le binding python
  • resize_save_values a été renommé conservativeResize, comme dans Eigen, comme ça on utilise directement la fonction Eigen
  • modification des CMakeLists.txt, un problème lors de la création des tests, les fichier étaient copier dans l'include et le linker de savait plus choisir entre la copy et l'originale
  • Dans la class Paving, j'ai déplacé l'initialisation du _tree. Il y avait un problème dans la création des Pavinginout et Pavingout: l'objet avait besoin d'être créer avant de faire le static_cast
  • dans PavingNode, il y avait un double pointeur shared_ptr entre le père et le fils, ce qui provoquait un problème à la destruction. J'ai ajouté un test pour vérifier ce problème.
  • dans cocdac2_matrices.h, j'ai changé ceci:
    template
    concept IsIntervalDomain = codac2::is_interval_based::value;
    //concept IsIntervalDomain = std::is_same_v<Scalar,codac2::Interval>;
    Avec le Scalar, ça permet d'avoir une compatibilité avec le type Affine que je veux intégrer, sans rien changer au comportement actuelle.
  • j'ai également améliorer le Approx
  • plein d'autres warnings mineurs...
    ..

…tation of the slice method in SlicedTube was returning a reference to a Slice object that was created from a shared pointer to a TSlice object. However, the shared pointer was being created with a custom deleter that did nothing, which meant that the TSlice object was never deleted, leading to a memory leak.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant