PointCloudLibrary/pcl

[custom] Remove empty destructors and constructors

Open

#4 283 ouverte le 20 juil. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (4 506 forks)batch import
effort: lowgood first issuekind: todo

Métriques du dépôt

Stars
 (9 023 stars)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

Description

This issue is to split PR #4249 into smaller pieces.

Per @SergioRAgostinho

With destructors I use the following policy:

if your class defines virtual methods which it defines (literally defining, not overriding), I explicitly default it. The compiler throws warnings telling if of your class has virtual methods but the destructor is not virtual.

If a class has empty / default destructors I just remove those lines of code. If the destructor is not implicitly generated you'll get a compiler error, so there's no chance you'll mess up.

For constructors: remove all default constructors unless there's a user defined (non-default) constructor.

From @kunaltyagi: https://godbolt.org/z/8f99dG

Guide contributeur

Direction de recherche
Recherchez les classes avec des destructeurs vides ou des constructeurs par défaut et supprimez les en suivant la politique : pour les classes avec des méthodes virtuelles, définissez explicitement le destructeur par défaut ; sinon, supprimez le. Pour les constructeurs, supprimez les constructeurs par défaut sauf s'il existe un constructeur non défini par l'utilisateur. Utilisez le lien godbolt pour vérifier le comportement.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85