Private Static consts
ClosedPublic

Authored by rizzitello on Nov 25 2018, 2:55 AM.

Diff Detail

Repository
R232 AtCore
Branch
staticPrivate
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5364
Build 5382: arc lint + arc unit
rizzitello requested review of this revision.Nov 25 2018, 2:55 AM
rizzitello created this revision.
rizzitello edited the summary of this revision. (Show Details)Nov 25 2018, 2:56 AM
rizzitello added a project: Atelier: AtCore.
rizzitello added a subscriber: Atelier: AtCore.
rizzitello updated this revision to Diff 46173.Nov 25 2018, 3:08 AM
rizzitello edited the summary of this revision. (Show Details)
  • Fix variable names
laysrodrigues accepted this revision.Nov 25 2018, 2:25 PM

looks good to me!

This revision is now accepted and ready to land.Nov 25 2018, 2:25 PM
patrickelectric requested changes to this revision.Nov 25 2018, 4:38 PM
patrickelectric added inline comments.
src/core/temperature.cpp
41

alphabetic order

57

you can use auto here.

111–113

msgString = QString::fromLatin1(msg) to avoid the usage of fromLatin1

This revision now requires changes to proceed.Nov 25 2018, 4:38 PM
rizzitello updated this revision to Diff 46215.Nov 25 2018, 5:33 PM
rizzitello marked 3 inline comments as done.
  • Patrick Suggestions
rizzitello added inline comments.Nov 25 2018, 5:34 PM
src/core/temperature.cpp
57

It will not build if auto is used. Compiler complains about redefined variable.

patrickelectric accepted this revision.Nov 26 2018, 11:16 PM
This revision is now accepted and ready to land.Nov 26 2018, 11:16 PM
laysrodrigues accepted this revision.Nov 27 2018, 2:06 PM
rizzitello closed this revision.Nov 27 2018, 8:57 PM