Comment Search

Done

Blood flow can be negative, hence, it is handled.

Is the payload length constrained by its definition or it can accidentally become negative?

Done.

RESOLVED IN CODE WALKTHROUGH.

Remove extra line.

Why so much space?

initRTC() only sets the self test to start so it does not have to be in the beginning.

I think the macro does the ';' so you don't need one.

Length is range checked in the above else if statement.

Magic number?

I believe these cases need indentation

RESOLVED IN CODE WALKTHROUGH.

Fixed

What CONST is this 3? Replace with LETTER_CONST and centralize all of them on top of code.

Done.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Move the local variable into the scope.

RESOLVED in CODE WALKTHROUGH.

Files has been modified many times manuallu on my branch as you asked a week or toow before 2020 and by Dara's script on master on 2019 and 2020.
It might be me merging files from my branch to master with 2019, since I think his script is running only on master branches.

Files has been modified many times manuallu on my branch as you asked a week or toow before 2020 and by Dara's script on master on 2019 and 2020.
It might be me merging files from my branch to master with 2019, since I think his script is running only on master branches.

Files has been modified many times manuallu on my branch as you asked a week or toow before 2020 and by Dara's script on master on 2019 and 2020.
It might be me merging files from my branch to master with 2019, since I think his script is running only on master branches.