Format all top-level source folders#4134
Conversation
| Checksum(const U32 value); | ||
|
|
||
| //! Copy a Checksum object. | ||
| Checksum(const Checksum& original); |
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning
| struct RandomizeRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| RandomizeRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct SerializeOkRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| SerializeOkRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct SerializeOverflowRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| SerializeOverflowRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct PeekOkRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| PeekOkRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct PeekBadRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| PeekBadRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct RotateOkRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| RotateOkRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| struct RotateBadRule : public STest::Rule<MockTypes::CircularState> { | ||
| // Constructor | ||
| RotateBadRule(const char* const name); | ||
|
|
Check warning
Code scanning / CppCheck
Single-parameter constructors should be marked explicit. Warning test
| // set new rate, replenish quickly | ||
| while (bucket.trigger(Fw::Time(0,0))); | ||
| while (bucket.trigger(Fw::Time(0, 0))) | ||
| ; |
Check warning
Code scanning / CppCheck
Line contains only semicolon. If this should be an empty statement, use {} instead. Warning test
There was a problem hiding this comment.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
Change Description
Fixes #3996
Adds last couple top-level folders formatting and CI checks.
NOTE: I did not add STest as it is already self-consistent.... but maybe we should. To be discussed.