reduce scope of pointer variables #2
Fechado
6 anos atrás abertas por Ghost
·
3 comentários
Sem etiqueta
Sem responsável
2 participante(s)
Notificações
Data limite
Data limite não informada.
Dependências
Nenhuma dependência definida.
Referência: TDE/yakuake#2
Carregando…
Referência em uma nova issue
Ainda não há conteúdo.
Excluir Branch '%!s(<nil>)'
A exclusão de um branch é permanente. Isto NÃO PODERÁ ser desfeito. Continuar?
According to cppcheck the scope of 'item' and 'skin' in "yakuake/yakuake/src/skin_settings.cpp" could be reduced:
[skin_settings.cpp:151]: (style) The scope of the variable 'item' can be reduced.
[skin_settings.cpp:418]: (style) The scope of the variable 'skin' can be reduced.
I suggest 'item' to be moved in the "while (it2.current())" scope.
and
'skin' to be moved in the while "(it.current())" scope.
What do you think about?
Yes, it sounds like a good idea. Please create PR and we will merge, so commit is with your name 😄
one more thing: in the commit message please make sure to add something like "This resolves issue#2", so that gitea can pick it up and automatically link issue and commit in code.
Resolved by PR #3