Atelier Clazy-tidy

Authored by rizzitello on Nov 23 2018, 4:00 AM.

Description

Atelier Clazy-tidy

Summary:
Resolve Clang-tidy / Clazy "issues" They include

  • QStringLiteral is slow for empty strings so use QString("")
  • More use of auto to cut back on Object *= new object...
  • more things with parents
  • Use default deconstuctors
  • Initialization of variables without defaults.
  • Const Refs to avoid extra copies
  • &More things

Reviewers: laysrodrigues, patrickelectric, tcanabrava

Reviewed By: laysrodrigues, patrickelectric, tcanabrava

Subscribers: Atelier

Tags: Atelier

Differential Revision: https://phabricator.kde.org/D17106

Merged Changes

CommitAuthorDetailsCommitted
fc590f2aae9eD17106rizzitello
Use default deconstuctor Set parents Automaticly select top entry Fix typo… 
Dec 14 2018
8034f7bbad64D17106rizzitello
Use auto Remove else after return Initialize var without defaults Don't use… 
Dec 14 2018
fe6f950cd00cD17106rizzitello
checkTemperature Show index only for extruder's temperatures. 
Dec 14 2018
111eff846378D17106rizzitello
Use auto Initialize vars without defaults remove else after return Set… 
Dec 14 2018
a0e925a4af49D17106rizzitello
Else after return Use R to reduce special char treatment Use default… 
Dec 14 2018
d8b5b03ce571D17106rizzitello
Use auto 
Dec 14 2018
d189a28cecccD17106rizzitello
Using to replace typedef Set Parent Use default deconstructor Signed-off-by… 
Dec 14 2018
60670232b4dfD17106rizzitello
Use auto Const Ref file Use = default for deconstructor 
Dec 14 2018
ddfea1aa56ddD17106rizzitello
Const Ref for name Use default deconstructor Set Parents 
Dec 14 2018
9f7afcc7c7a4D17106rizzitello
Use auto Initialize vars without defaults Use = default deconstructor Set… 
Dec 14 2018
0298d46ddcb1D17106rizzitello
Remove else after return Remove Unneeded Debug calls Use = default for… 
Dec 14 2018
4f98526c1110D17106rizzitello
Use = default for deconstructor 
Dec 14 2018
ee5cb7237a24D17106rizzitello
Use = default for deconstructor 
Dec 14 2018
fecf7dde7746D17106rizzitello
Use = default for deconstructor 
Dec 14 2018
cfe0234d9ffbD17106rizzitello
Reduce qApp calls Use QString for empty QString Signed-off-by: Chris Rizzitello… 
Dec 14 2018
64cf5f11905fD17106rizzitello
Use = for deconstructor 
Dec 14 2018
b446c8547e38D17106rizzitello
Use = for deconstructor 
Dec 14 2018
d1876f5b2e17D17106rizzitello
Use = for deconstructor Initialize vars without defaults Set parents Signed… 
Dec 14 2018
ecad265eecb0D17106rizzitello
Use = for deconstructor 
Dec 14 2018
8e95b60ac30eD17106rizzitello
Remove Namespace Use = default for deconstructor 
Dec 14 2018