RESOLVED.
RESOLVED in CODE WALKTHROUGH.
Done.
Done
Should this not be commented out since there is a ifdef guarding it?
Should this be an else instead of else if?
With QVERIFY commented out, this isn't testing anything
Blank line is in wrong place.
data parameter is F32 so this else branch seems pointless.
Fixed.
Why is this module both in fwcommon and firmware at the same time?
1. Why is indentation of } off? 2. Also no extra line before }
The percent sign should go after the number, not before it
RESOLVED
Sure, this is a play space, the V&V ones, however, will be much more controlled. But, I would still recommend using the name of module or epic to avoid including personal names.
Don't you need the first element in each tuple?
1. inletTemperatrue? 2. Variable declared but never used?
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.
