PointCloudLibrary/pcl

[custom] Remove empty destructors and constructors

Open

#4283 aperta il 20 lug 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
effort: lowgood first issuekind: todo

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Cerca le classi con distruttori vuoti o costruttori predefiniti e rimuovili seguendo la policy: per le classi con metodi virtuali, imposta esplicitamente il distruttore come predefinito; altrimenti rimuovilo. Per i costruttori, rimuovi i costruttori predefiniti a meno che non esista un costruttore non predefinito definito dall'utente. Usa il link godbolt per verificare il comportamento.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85