Repair this code

The starter clamp.c fails to build on three separate diagnostics: clear all three so clamp returns v held inside the bounds lo and hi.

Steps

• clamp receives three int parameters: the value v, the low bound lo, and the high bound hi.
• Return lo when v is below lo, hi when v is above hi, and v itself when it already sits between them.
• Both bounds are inclusive, so a v equal to lo or equal to hi comes back unchanged.
• The bounds are taken as ordered. When lo is greater than hi they are inverted, and clamp returns lo for every v.
• A lo equal to hi is a range of one value: every v maps to it.
• Negative bounds, a range that straddles zero, -2147483648 and 2147483647 are all ordinary input. The function only compares values, so nothing overflows.
• Your first run stops at the compile stage instead of showing a red test. The grader builds with -Wextra -Werror, which turns each of the three warnings into an error.
• The three are an unused parameter, an unused local variable, and a comparison between a signed and an unsigned operand. gcc reports them all in one pass, so read the list top to bottom and rebuild after each fix.
• Read a diagnostic before you silence it. Adding (void)hi; clears that line of the list and still leaves the tests red: hi is unused because the branch that needs it was never written.
• The signedness warning is not cosmetic either. Comparing an int against an unsigned int converts the int first, which breaks every negative bound. Keep both sides of that comparison signed.

Repair this code

The starter clamp.c fails to build on three separate diagnostics: clear all three so clamp returns v held inside the bounds lo and hi.

Steps

• clamp receives three int parameters: the value v, the low bound lo, and the high bound hi.
• Return lo when v is below lo, hi when v is above hi, and v itself when it already sits between them.
• Both bounds are inclusive, so a v equal to lo or equal to hi comes back unchanged.
• The bounds are taken as ordered. When lo is greater than hi they are inverted, and clamp returns lo for every v.
• A lo equal to hi is a range of one value: every v maps to it.
• Negative bounds, a range that straddles zero, -2147483648 and 2147483647 are all ordinary input. The function only compares values, so nothing overflows.
• Your first run stops at the compile stage instead of showing a red test. The grader builds with -Wextra -Werror, which turns each of the three warnings into an error.
• The three are an unused parameter, an unused local variable, and a comparison between a signed and an unsigned operand. gcc reports them all in one pass, so read the list top to bottom and rebuild after each fix.
• Read a diagnostic before you silence it. Adding (void)hi; clears that line of the list and still leaves the tests red: hi is unused because the branch that needs it was never written.
• The signedness warning is not cosmetic either. Comparing an int against an unsigned int converts the int first, which breaks every negative bound. Keep both sides of that comparison signed.