PointCloudLibrary/pcl

[custom] Remove empty destructors and constructors

Open

#4.283 aberto em 20 de jul. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
effort: lowgood first issuekind: todo

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Procure por classes com destrutores vazios ou construtores padrão e remova os seguindo a política: para classes com métodos virtuais, defina explicitamente o destrutor como padrão; caso contrário, remova o. Para construtores, remova construtores padrão a menos que haja um construtor não padrão definido pelo usuário. Use o link godbolt para verificar o comportamento.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85