fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove blank line.

Remove blank line.

Use enums instead of literals.

Use enums instead of literals.

Don't we have an enum for these values?

Don't we have an enum for these values?

Is there more to be done here (TODO)?

Is there more to be done here (TODO)?

I think these variable declarations should stay here. Why move to function?

I think these variable declarations should stay here. Why move to function?

This is HD f/w. Why are you looking at DG mode?

This is HD f/w. Why are you looking at DG mode?

Why aren't these declared at top of file and initialized in init function like everything else.

Why aren't these declared at top of file and initialized in init function like everything else.

Why is this in two places? And why set every 10ms when it appears to be constant?

Why is this in two places? And why set every 10ms when it appears to be constant?

This doesn't seem to ever change. Why not just initialize this value once on reset?

This doesn't seem to ever change. Why not just initialize this value once on reset?

Make U32.

Make U32.

Declarations before code.

Declarations before code.

Should be != TRUE?

Should be != TRUE?

loadcells[] is an output in this function.

loadcells[] is an output in this function.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Keep going. Now 95 is wrong. Need comments to align with actual values throughout.

Keep going. Now 95 is wrong. Need comments to align with actual values throughout.

Done

Done

Then the one above with 100 comment is wrong.

Then the one above with 100 comment is wrong.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

Done

Done

Is this a big risk? At this time I am not too concerned about it. If you think it should be addressed, I can make changes.

Is this a big risk? At this time I am not too concerned about it. If you think it should be addressed, I can make changes.

Done

Done

Done

Done

This is 105.

This is 105.